diff for duplicates of <20170121050445.GG15269@mwanda> diff --git a/a/1.txt b/N1/1.txt index 9ad5c27..bb07e01 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,7 @@ The issue here is that in orangefs_bufmap_alloc() we do: - bufmap->buffer_index_array kzalloc(DIV_ROUND_UP(bufmap->desc_count, BITS_PER_LONG), GFP_KERNEL); + bufmap->buffer_index_array = + kzalloc(DIV_ROUND_UP(bufmap->desc_count, BITS_PER_LONG), GFP_KERNEL); If we choose a bufmap->desc_count like -31 then it means the DIV_ROUND_UP ends up having an integer overflow. The result is that diff --git a/a/content_digest b/N1/content_digest index 4a494cd..711ce5f 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,6 +1,6 @@ "From\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Subject\0[patch] orangefs: silence harmless integer overflow warning\0" - "Date\0Sat, 21 Jan 2017 05:04:45 +0000\0" + "Date\0Sat, 21 Jan 2017 08:04:45 +0300\0" "To\0Mike Marshall <hubcap@omnibond.com>\0" "Cc\0linux-kernel@vger.kernel.org" " kernel-janitors@vger.kernel.org\0" @@ -8,7 +8,8 @@ "b\0" "The issue here is that in orangefs_bufmap_alloc() we do:\n" "\n" - "\tbufmap->buffer_index_array \t\tkzalloc(DIV_ROUND_UP(bufmap->desc_count, BITS_PER_LONG), GFP_KERNEL);\n" + "\tbufmap->buffer_index_array =\n" + "\t\tkzalloc(DIV_ROUND_UP(bufmap->desc_count, BITS_PER_LONG), GFP_KERNEL);\n" "\n" "If we choose a bufmap->desc_count like -31 then it means the\n" "DIV_ROUND_UP ends up having an integer overflow. The result is that\n" @@ -41,4 +42,4 @@ " \t * sanity check alignment and size of buffer that caller wants to\n" " \t * work with" -635dddab53e83a7b11172d3789e77ee9e428f126f23ee3a30bbcc7e440709e4a +1741900eb7dc5419670c9c2d2157a8c815a500a28d206dd99a23bdcf83594f62
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.