diff options
author | Bob Peterson <rpeterso@redhat.com> | 2008-01-28 17:35:13 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2008-03-31 05:40:05 -0400 |
commit | 3ad62e87cd38817361e165cf4ad496ab76e19e81 (patch) | |
tree | b094e3244239d67d1d1c72abac50a1ba792aaa6a | |
parent | 048786f1e6042022a8fb2035157a8c8c3a82a4f2 (diff) |
[GFS2] Plug an unlikely leak
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
-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, | |||
404 | blkno = be64_to_cpu(*(__be64 *)(bh->b_data + offset)); | 404 | blkno = be64_to_cpu(*(__be64 *)(bh->b_data + offset)); |
405 | 405 | ||
406 | error = gfs2_revoke_add(sdp, blkno, start); | 406 | error = gfs2_revoke_add(sdp, blkno, start); |
407 | if (error < 0) | 407 | if (error < 0) { |
408 | brelse(bh); | ||
408 | return error; | 409 | return error; |
410 | } | ||
409 | else if (error) | 411 | else if (error) |
410 | sdp->sd_found_revokes++; | 412 | sdp->sd_found_revokes++; |
411 | 413 | ||