diff options
| author | Darren Jenkins <darrenrjenkins@gmail.com> | 2008-07-23 13:55:59 -0400 |
|---|---|---|
| committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-07-23 13:55:59 -0400 |
| commit | 4b5075050072ed7c4730ee547ae80117c4fa8513 (patch) | |
| tree | f980e483566fa43f6e93a2fbbbcfbe6553f4e05d /drivers/ide | |
| parent | 8a69580e1ea9516caada5eed202afd39546e9809 (diff) | |
drivers/ide/ide-tape.c: remove double kfree
Coverity CID: 2336 USE_AFTER_FREE
drivers/ide/ide-tape.c ide_tape_kfree_buffer() double free's a pointer
(hint freed by an alias in first outer loop).
This patch simply removes the superfluous kfree().
Signed-off-by: Darren Jenkins <darrenrjenkins@gmailcom>
Acked-by: Borislav Petkov <petkovbb@gmail.com>
Cc: gadio@netvision.net.il
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide')
| -rw-r--r-- | drivers/ide/ide-tape.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c index 0af128826f1e..663671f69f88 100644 --- a/drivers/ide/ide-tape.c +++ b/drivers/ide/ide-tape.c | |||
| @@ -585,7 +585,6 @@ static void ide_tape_kfree_buffer(idetape_tape_t *tape) | |||
| 585 | bh = bh->b_reqnext; | 585 | bh = bh->b_reqnext; |
| 586 | kfree(prev_bh); | 586 | kfree(prev_bh); |
| 587 | } | 587 | } |
| 588 | kfree(tape->merge_bh); | ||
| 589 | } | 588 | } |
| 590 | 589 | ||
| 591 | static int idetape_end_request(ide_drive_t *drive, int uptodate, int nr_sects) | 590 | static int idetape_end_request(ide_drive_t *drive, int uptodate, int nr_sects) |