diff options
| author | Bob Peterson <rpeterso@redhat.com> | 2008-01-28 23:35:13 +0100 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2008-03-31 11:40:05 +0200 |
| commit | 3ad62e87cd38817361e165cf4ad496ab76e19e81 (patch) | |
| tree | b094e3244239d67d1d1c72abac50a1ba792aaa6a /fs/gfs2/lops.c | |
| parent | [GFS2] make gfs2_glock_hold() static (diff) | |
| download | linux-3ad62e87cd38817361e165cf4ad496ab76e19e81.tar.xz linux-3ad62e87cd38817361e165cf4ad496ab76e19e81.zip | |
[GFS2] Plug an unlikely leak
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/lops.c')
| -rw-r--r-- | fs/gfs2/lops.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/gfs2/lops.c b/fs/gfs2/lops.c index 71387372c883..4390f6f4047d 100644 --- a/fs/gfs2/lops.c +++ b/fs/gfs2/lops.c @@ -404,8 +404,10 @@ static int revoke_lo_scan_elements(struct gfs2_jdesc *jd, unsigned int start, blkno = be64_to_cpu(*(__be64 *)(bh->b_data + offset)); error = gfs2_revoke_add(sdp, blkno, start); - if (error < 0) + if (error < 0) { + brelse(bh); return error; + } else if (error) sdp->sd_found_revokes++; |