diff for duplicates of <20170926201105.126166-4-ebiggers3@gmail.com> diff --git a/a/1.txt b/N1/1.txt index f729868..1cb39ff 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -4,7 +4,8 @@ In key_validate(), load the flags and expiry time once atomically, since these can change concurrently if key_validate() is called without the key semaphore held. And we don't want to get inconsistent results if a variable is referenced multiple times. For example, key->expiry was -referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >key->expiry)', making it theoretically possible to see a spurious +referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >= +key->expiry)', making it theoretically possible to see a spurious EKEYEXPIRED while the expiration time was being removed, i.e. set to 0. Signed-off-by: Eric Biggers <ebiggers@google.com> @@ -40,3 +41,8 @@ index 732cc0beffdf..a72b4dd70c8a 100644 -- 2.14.1.992.g2c7b836f3a-goog + +-- +To unsubscribe from this list: send the line "unsubscribe linux-security-module" in +the body of a message to majordomo at vger.kernel.org +More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 7d57a4a..a90d816 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\020170926201105.126166-1-ebiggers3@gmail.com\0" - "From\0Eric Biggers <ebiggers3@gmail.com>\0" + "From\0ebiggers3@gmail.com (Eric Biggers)\0" "Subject\0[PATCH v2 3/6] KEYS: load key flags and expiry time atomically in key_validate()\0" - "Date\0Tue, 26 Sep 2017 20:11:02 +0000\0" + "Date\0Tue, 26 Sep 2017 13:11:02 -0700\0" "To\0linux-security-module@vger.kernel.org\0" "\00:1\0" "b\0" @@ -11,7 +11,8 @@ "these can change concurrently if key_validate() is called without the\n" "key semaphore held. And we don't want to get inconsistent results if a\n" "variable is referenced multiple times. For example, key->expiry was\n" - "referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >key->expiry)', making it theoretically possible to see a spurious\n" + "referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >=\n" + "key->expiry)', making it theoretically possible to see a spurious\n" "EKEYEXPIRED while the expiration time was being removed, i.e. set to 0.\n" "\n" "Signed-off-by: Eric Biggers <ebiggers@google.com>\n" @@ -46,6 +47,11 @@ " \t}\n" " \n" "-- \n" - 2.14.1.992.g2c7b836f3a-goog + "2.14.1.992.g2c7b836f3a-goog\n" + "\n" + "--\n" + "To unsubscribe from this list: send the line \"unsubscribe linux-security-module\" in\n" + "the body of a message to majordomo at vger.kernel.org\n" + More majordomo info at http://vger.kernel.org/majordomo-info.html -d55d209170d8f8276c5eb4273d36ab0800101eefee65ea5572d87804a46563e1 +5741e336e2d6d5fa2226160d46513a12a7930c3b4f3d0df814df23a696e3547a
diff --git a/a/1.txt b/N2/1.txt index f729868..b6b84a3 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -4,7 +4,8 @@ In key_validate(), load the flags and expiry time once atomically, since these can change concurrently if key_validate() is called without the key semaphore held. And we don't want to get inconsistent results if a variable is referenced multiple times. For example, key->expiry was -referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >key->expiry)', making it theoretically possible to see a spurious +referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >= +key->expiry)', making it theoretically possible to see a spurious EKEYEXPIRED while the expiration time was being removed, i.e. set to 0. Signed-off-by: Eric Biggers <ebiggers@google.com> diff --git a/a/content_digest b/N2/content_digest index 7d57a4a..89e704d 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,8 +1,13 @@ "ref\020170926201105.126166-1-ebiggers3@gmail.com\0" "From\0Eric Biggers <ebiggers3@gmail.com>\0" "Subject\0[PATCH v2 3/6] KEYS: load key flags and expiry time atomically in key_validate()\0" - "Date\0Tue, 26 Sep 2017 20:11:02 +0000\0" - "To\0linux-security-module@vger.kernel.org\0" + "Date\0Tue, 26 Sep 2017 13:11:02 -0700\0" + "To\0keyrings@vger.kernel.org\0" + "Cc\0David Howells <dhowells@redhat.com>" + Michael Halcrow <mhalcrow@google.com> + linux-security-module@vger.kernel.org + linux-kernel@vger.kernel.org + " Eric Biggers <ebiggers@google.com>\0" "\00:1\0" "b\0" "From: Eric Biggers <ebiggers@google.com>\n" @@ -11,7 +16,8 @@ "these can change concurrently if key_validate() is called without the\n" "key semaphore held. And we don't want to get inconsistent results if a\n" "variable is referenced multiple times. For example, key->expiry was\n" - "referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >key->expiry)', making it theoretically possible to see a spurious\n" + "referenced in both 'if (key->expiry)' and in 'if (now.tv_sec >=\n" + "key->expiry)', making it theoretically possible to see a spurious\n" "EKEYEXPIRED while the expiration time was being removed, i.e. set to 0.\n" "\n" "Signed-off-by: Eric Biggers <ebiggers@google.com>\n" @@ -48,4 +54,4 @@ "-- \n" 2.14.1.992.g2c7b836f3a-goog -d55d209170d8f8276c5eb4273d36ab0800101eefee65ea5572d87804a46563e1 +301d2761eabdb8fae6dcf0f66e07c44afdf38d2c5114e0800b71334dd8440659
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.