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

2007-07-19 Thread James Bottomley
On Thu, 2007-07-19 at 13:44 -0400, James Smart wrote: > Florin, > > Thanks for the effort to create this patch. Turns out we have already > fixed this, and have it included in our 8.2.2 patch set, which we will be > posting within the next week. The 2.6.23 merge window is estimated to close on

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

2007-07-19 Thread James Smart
Florin, Thanks for the effort to create this patch. Turns out we have already fixed this, and have it included in our 8.2.2 patch set, which we will be posting within the next week. So James, don't pick this patch up, take the larger patch set instead. Thanks. -- james s Florin Malita

[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_sli.c

[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_sli.c

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

2007-07-19 Thread James Smart
Florin, Thanks for the effort to create this patch. Turns out we have already fixed this, and have it included in our 8.2.2 patch set, which we will be posting within the next week. So James, don't pick this patch up, take the larger patch set instead. Thanks. -- james s Florin Malita

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

2007-07-19 Thread James Bottomley
On Thu, 2007-07-19 at 13:44 -0400, James Smart wrote: Florin, Thanks for the effort to create this patch. Turns out we have already fixed this, and have it included in our 8.2.2 patch set, which we will be posting within the next week. The 2.6.23 merge window is estimated to close on