aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSilvio Gissi <sifonsec@amazon.com>2024-03-15 15:05:39 -0400
committerJarkko Sakkinen <jarkko@kernel.org>2024-03-26 16:24:53 +0200
commit6999f8229e5998e8286e6a960779b6c202d878da (patch)
tree0e80fadd775a96a837c9c3d25adac6c8bd8cda24
parent019e41d1bee9bb730dfd1ccecabdb254e5c3b6cc (diff)
downloadlinux-tpmdd-6999f8229e5998e8286e6a960779b6c202d878da.tar.gz
keys: Fix overwrite of key expiration on instantiation
Notice: this object is not reachable from any branch.
tpmdd-v6.9-rc2
The expiry time of a key is unconditionally overwritten during instantiation, defaulting to turn it permanent. This causes a problem for DNS resolution as the expiration set by user-space is overwritten to TIME64_MAX, disabling further DNS updates. Fix this by restoring the condition that key_set_expiry is only called when the pre-parser sets a specific expiry. Fixes: 39299bdd2546 ("keys, dns: Allow key types (eg. DNS) to be reclaimed immediately on expiry") Signed-off-by: Silvio Gissi <sifonsec@amazon.com> cc: David Howells <dhowells@redhat.com> cc: Hazem Mohamed Abuelfotoh <abuehaze@amazon.com> cc: linux-afs@lists.infradead.org cc: linux-cifs@vger.kernel.org cc: keyrings@vger.kernel.org cc: netdev@vger.kernel.org cc: stable@vger.kernel.org Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
Notice: this object is not reachable from any branch.
-rw-r--r--security/keys/key.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/security/keys/key.c b/security/keys/key.c
index 2a9a769e795e1..3d7d185019d30 100644
--- a/security/keys/key.c
+++ b/security/keys/key.c
@@ -465,7 +465,8 @@ static int __key_instantiate_and_link(struct key *key,
if (authkey)
key_invalidate(authkey);
- key_set_expiry(key, prep->expiry);
+ if (prep->expiry != TIME64_MAX)
+ key_set_expiry(key, prep->expiry);
}
}