diff for duplicates of <20171006121939.GC3314@codeblueprint.co.uk> diff --git a/a/1.txt b/N1/1.txt index 917b9f4..e99e797 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -14,7 +14,7 @@ On Sat, 30 Sep, at 11:17:32AM, Dan Carpenter wrote: > if (copy_from_user(&qcaps, qcaps_user, sizeof(qcaps))) > return -EFAULT; > -> + if (qcaps.capsule_count = ULONG_MAX) +> + if (qcaps.capsule_count == ULONG_MAX) > + return -EINVAL; > + > capsules = kcalloc(qcaps.capsule_count + 1, diff --git a/a/content_digest b/N1/content_digest index bb0b25a..069bdf7 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,7 +1,7 @@ "ref\020170930081732.x6g4b2mwxwt5xzql@mwanda\0" "From\0Matt Fleming <matt@codeblueprint.co.uk>\0" "Subject\0Re: [PATCH] efi/efi_test: Prevent an Oops in efi_runtime_query_capsulecaps()\0" - "Date\0Fri, 06 Oct 2017 12:19:39 +0000\0" + "Date\0Fri, 6 Oct 2017 13:19:39 +0100\0" "To\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Cc\0Ivan Hu <ivan.hu@canonical.com>" Ard Biesheuvel <ard.biesheuvel@linaro.org> @@ -25,7 +25,7 @@ "> \tif (copy_from_user(&qcaps, qcaps_user, sizeof(qcaps)))\n" "> \t\treturn -EFAULT;\n" "> \n" - "> +\tif (qcaps.capsule_count = ULONG_MAX)\n" + "> +\tif (qcaps.capsule_count == ULONG_MAX)\n" "> +\t\treturn -EINVAL;\n" "> +\n" "> \tcapsules = kcalloc(qcaps.capsule_count + 1,\n" @@ -34,4 +34,4 @@ "\n" This looks OK to me. Ivan? -5b61453945a769be48b2e51fac7457cdcad3f896caf9aaac4f6a2348a8767eb1 +011a2041db27c3ee67be00b51318f16f6312a33e5043eeed029c48c4ff117746
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.