[PATCH] V4L: possible leak in em28xx_init_isoc

2007-10-15 Thread Florin Malita
Coverity (CID 1929) spotted the following: if a transfer buffer allocation fails, the last allocated urb is leaked (it hasn't been stored in dev->urb[] yet so em28xx_uninit_isoc misses it). The patch also includes a small typo fix. Signed-off-by: Florin Malita <[EMAIL PROTECTED]>

[PATCH] mmc: possible leak in mmc_read_ext_csd

2007-10-13 Thread Florin Malita
The exception path associated with an invalid ext_csd_struct returns without freeing ext_csd. Coverity CID 1909. Signed-off-by: Florin Malita --- drivers/mmc/core/mmc.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c

[PATCH] Double-free in cx23885_initdev

2007-10-13 Thread Florin Malita
. Coverity CID 1922. Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- drivers/media/video/cx23885/cx23885-core.c |6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/media/video/cx23885/cx23885-core.c b/drivers/media/video/cx23885/cx23885-core.c index af16505..3

[PATCH] MTD: potential leak in rfd_ftl_add_mtd

2007-07-19 Thread Florin Malita
This fixes a leak in the !mtd->erasesize error path (Coverity 1765). Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- drivers/mtd/rfd_ftl.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mtd/rfd_ftl.c b/drivers/mtd/rfd_ftl.c index d4b1ba8..006c03a 100644 --- a/dr

[PATCH] infiniband mlx4: potential leaks in __mlx4_ib_modify_qp

2007-07-19 Thread Florin Malita
Temporarily allocated struct mlx4_qp_context *context is leaked by several error paths. The patch takes advantage of the return value 'err' being preinitialized to -EINVAL. Spotted by Coverity (1768). Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- drivers/infiniband/hw/

[PATCH] UBI: potential leak in ubi_scan_erase_peb

2007-07-19 Thread Florin Malita
Coverity (1769) found the following problem: if the erase counter overflow check triggers, ec_hdr is leaked. Moving the allocation after the overflow check should take care of it. Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- drivers/mtd/ubi/scan.c |8 1 file chan

[PATCH] [scsi] lpfc: potential leak in lpfc_sli_hbqbuf_fill_hbqs

2007-07-19 Thread Florin Malita
If lpfc_hbq_alloc() fails, an hbq_buffer is leaked. Found by Coverity (1775). Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- drivers/scsi/lpfc/lpfc_sli.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc

[PATCH] mos7720, mos7840: remove redundant urb check

2007-07-16 Thread Florin Malita
d-off-by: Florin Malita <[EMAIL PROTECTED]> --- drivers/usb/serial/mos7720.c |5 - drivers/usb/serial/mos7840.c | 19 --- 2 files changed, 24 deletions(-) diff --git a/drivers/usb/serial/mos7720.c b/drivers/usb/serial/mos7720.c index 231b584..01e811b 100644 --- a/

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-15 Thread Florin Malita
On 06/15/2007 02:30 PM, Michael Poole wrote: Florin Malita writes: On 06/15/2007 12:18 PM, Michael Poole wrote: Yes. If I cut a book in half and store the halves separately, does the second half become an independent work? Except in this case you're not touching the bo

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-15 Thread Florin Malita
On 06/15/2007 12:18 PM, Michael Poole wrote: Florin Malita writes: On 06/15/2007 10:56 AM, Michael Poole wrote: The GPL cares about the key used to generate an integral part of the executable form of the GPLed work. GLPv2 doesn't: why do you think the digital signature

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-15 Thread Florin Malita
On 06/15/2007 10:33 AM, Nicolas Mailhot wrote: Le Ven 15 juin 2007 15:41, Jesper Juhl a écrit : But the only thing that *actually* matters is what the license text *says*. It doesn't matter what the authors of the license text intended Judges would disagree there I very much doubt

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-15 Thread Florin Malita
On 06/15/2007 10:56 AM, Michael Poole wrote: The GPL cares about the key used to generate an integral part of the executable form of the GPLed work. GLPv2 doesn't: why do you think the digital signature is an integral part of the executable? It can be a totally separate blob, distributed vi

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-14 Thread Florin Malita
On 06/14/2007 05:39 PM, Alexandre Oliva wrote: On Jun 14, 2007, Florin Malita <[EMAIL PROTECTED]> wrote: No, it's not: replacing does not create derivative work. Modification does. Thanks. Good point. This convinces me that this doesn't work as a legal argument und

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-14 Thread Florin Malita
On 06/14/2007 02:27 PM, Alexandre Oliva wrote: No, by this twisted logic Tivo *cannot* modify that particular copy any more than you can. They can modify *another* copy (just like you) and they can *replace* the copy in your device with the new version (unlike you). Again, replacing is one

Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3

2007-06-14 Thread Florin Malita
Alexandre Oliva wrote: On Jun 14, 2007, Daniel Hazelton <[EMAIL PROTECTED]> wrote: When they download the software, they get another copy, and they have a right to modify that copy. But you get the TiVO corporations copy of the software? Yes. The customer gets the copy that Ti

[PATCH] spi: potential memleak in spidev_ioctl

2007-05-16 Thread Florin Malita
'ioc' should be deallocated if __copy_from_user fails (found by Coverity - CID 1644). Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- spidev.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/spi/spidev.c b/drivers/spi/spidev.c index c0a6dce..2464f34 100644 -

Re: [PATCH] UBI: dereference after kfree in create_vtbl

2007-05-04 Thread Florin Malita
n't have to deal with allocation failures in an error path anymore - something like this (diff against the original code): Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- diff --git a/drivers/mtd/ubi/vtbl.c b/drivers/mtd/ubi/vtbl.c index b6fd6bb..2ad2d59 100644 --- a/drivers/mtd/ub

[PATCH] UBI: dereference after kfree in create_vtbl

2007-05-03 Thread Florin Malita
Coverity (CID 1614) spotted new_seb being dereferenced after kfree() in create_vtbl's write_error path. Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- vtbl.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/mtd/ubi/vtbl.c b/drivers/mtd

[PATCH] devpts: add fsnotify create event

2007-02-20 Thread Florin Malita
ify_create() trigger in devpts_pty_new(). Signed-off-by: Florin Malita <[EMAIL PROTECTED]> --- fs/devpts/inode.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/fs/devpts/inode.c b/fs/devpts/inode.c index 643e57b..06ef9a2 100644 --- a/fs/devpts/inode.c +++ b/f

Re: kernel 2.6 speed

2005-07-26 Thread Florin Malita
[EMAIL PROTECTED] wrote: Anything time stamping things it processes many of will call some sort of time function pretty often. Could happen frequently with certain classes of applications. Right, but if the timestamp granularity is coarse and there's no blocking call in between it makes no se

Re: kernel 2.6 speed

2005-07-24 Thread Florin Malita
On 7/24/05, Alan Cox <[EMAIL PROTECTED]> wrote: > time() isn't a hot > path in the real world. That's what you would expect but I've straced stuff calling gettimeofday() in huge bursts every other second. Obviously braindead stuff but so is "the real world" most of the time() ... :) - To unsubscri

Re: kernel 2.6 speed

2005-07-24 Thread Florin Malita
On 7/24/05, Ciprian <[EMAIL PROTECTED]> wrote: > test /= 10; > test *= 10; > test += 10; > test -= 10; You're not trying to benchmark the kernel with those arithmetic operations are you?! That's completely bogus, the kernel is not involved in any of that. As it has been already pointed out, the o