Re: New Defect(s) reported by Coverity Scan

2013-04-08 Thread Steven Rostedt
Removed other's Cc'd. On Mon, 2013-04-01 at 22:34 +, Scan Subscription wrote: > http://scan5.coverity.com:8080/sourcebrowser.htm?projectId=10063#mergedDefectId=102438 > > /kernel/trace/trace.c: 2431 ( var_compare_op) >2428 int ret; >2429 >2430 if (iter->ent == NU

New Defect(s) reported by Coverity Scan

2013-04-05 Thread Scan Subscription
Hi, Please find the latest report on new defect(s) that have been introduced to the Linux Kernel found with Coverity SCAN. Defect(s) Reported-by: Coverity Scan: ___ ** CID 996157: Logically dead code (DEADCODE) /net/mac80

Re: New Defect(s) reported by Coverity Scan

2013-04-01 Thread David Miller
Please take me off of the CC list for these reports. Thank you. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tu

New Defect(s) reported by Coverity Scan

2013-04-01 Thread Scan Subscription
Hi, Please find the latest report on new defect(s) that have been introduced to the Linux Kernel found with Coverity Scan. Defect(s) Reported-by: Coverity Scan: ___ ** CID 993740: Copy-paste error (COPY_PASTE_ERROR) /net/

Re: New Defect(s) reported by Coverity Scan

2013-02-13 Thread H. Peter Anvin
On 02/13/2013 03:22 PM, Borislav Petkov wrote: > On Wed, Feb 13, 2013 at 03:15:26PM -0800, H. Peter Anvin wrote: >> Not really, since exit() does that. We might want it for clarity's >> sake, though. > > Well, I would add a comment only then and teach the tool when parsing > exit() to know that -

Re: New Defect(s) reported by Coverity Scan

2013-02-13 Thread Borislav Petkov
On Wed, Feb 13, 2013 at 03:15:26PM -0800, H. Peter Anvin wrote: > Not really, since exit() does that. We might want it for clarity's > sake, though. Well, I would add a comment only then and teach the tool when parsing exit() to know that - no need for fixing perfectly good code only for a tool's

Re: New Defect(s) reported by Coverity Scan

2013-02-13 Thread H. Peter Anvin
On 02/13/2013 11:13 AM, Borislav Petkov wrote: > > and it might really need fclose() there for correctness' sake. > Not really, since exit() does that. We might want it for clarity's sake, though. -hpa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the b

Re: New Defect(s) reported by Coverity Scan

2013-02-13 Thread Borislav Petkov
On Wed, Feb 13, 2013 at 06:55:03PM +, Scan Subscription wrote: > Hi Boris > > There is no fclose(file) call in parse_zoffset() function > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=blob_plain;f=arch/x86/boot/tools/build.c;hb=HEAD > > > There is only one flcose call in th

RE: New Defect(s) reported by Coverity Scan

2013-02-13 Thread Scan Subscription
: New Defect(s) reported by Coverity Scan On Wed, Feb 13, 2013 at 05:40:11PM +, Scan Subscription wrote: > Hi, > > Please find the latest report on new defect(s) that have been introduced to > the Linux Kernel found with Coverity SCAN. > > > Defect(s) Repor

Re: New Defect(s) reported by Coverity Scan

2013-02-13 Thread Borislav Petkov
On Wed, Feb 13, 2013 at 05:40:11PM +, Scan Subscription wrote: > Hi, > > Please find the latest report on new defect(s) that have been introduced to > the Linux Kernel found with Coverity SCAN. > > > Defect(s) Reported-by: Coverity Scan: > __

New Defect(s) reported by Coverity Scan

2013-02-13 Thread Scan Subscription
Hi, Please find the latest report on new defect(s) that have been introduced to the Linux Kernel found with Coverity SCAN. Defect(s) Reported-by: Coverity Scan: ___ ** CID 971353: Resource leak (RESOURCE_LEAK) /arch/x86/bo

New Defect(s) reported by Coverity Scan

2013-01-29 Thread Scan Subscription
Hi, Please find the latest report on new defect(s) that have been introduced to the Linux Kernel found with Coverity SCAN. Defect(s) Reported-by: Coverity Scan: ___ ** CID 967920: Allocation too small for type (SIZECHECK)

[PATCH 2] Re: New Defect(s) reported by Coverity Scan

2012-12-31 Thread Nigel Cunningham
From 68e866b8eac534405ae16b79b7ffd9de05c11c67 Mon Sep 17 00:00:00 2001 From: Nigel Cunningham Date: Tue, 1 Jan 2013 13:50:22 +1100 Subject: [PATCH] Fix uninitialised variable in rbd_dev_probe_update_spec. The local variable ret can be used uninitialised in the error path if the kstrdup at line 2

[PATCH 3] Re: New Defect(s) reported by Coverity Scan

2012-12-31 Thread Nigel Cunningham
From b4a7ab768df17e1cda7d0ae8744e986215a644c3 Mon Sep 17 00:00:00 2001 From: Nigel Cunningham Date: Tue, 1 Jan 2013 13:53:51 +1100 Subject: [PATCH] Remove unused variable in rbd_dev_probe_update_spec. As an aside to the previous patch, remove the unused local variable reply_buf in that function.

[PATCH] Re: New Defect(s) reported by Coverity Scan

2012-12-31 Thread Nigel Cunningham
From b41864867464bfe0e2d114528bc9b39e2d9f546e Mon Sep 17 00:00:00 2001 From: Nigel Cunningham Date: Tue, 1 Jan 2013 13:03:50 +1100 Subject: [PATCH] Fix rbd use after free. This patch addresses Coverity #753114. The use of ceph_opts in rbd_add is currently confusing - there are three possible ou

New Defect(s) reported by Coverity Scan

2012-12-31 Thread Scan Subscription
Hi, Please find the latest report on new defect(s) that have been introduced to the Linux Kernel found with Coverity SCAN. Defect(s) Reported-by: Coverity Scan: ___ ** CID 753114: Use after free (USE_AFTER_FREE) /drivers/

New Defect(s) reported by Coverity Scan

2012-08-22 Thread Scan Subscription
Hi, Please find the latest report on new defect(s) that have been introduced to the Linux Kernel found with Coverity SCAN. Defect(s) Reported-by: Coverity Scan: ___ ** CID 716812: Unchecked return value (CHECKED_RETURN) /