From: Andrew Deason <adeason@sinenomine.net>
Date: Tue, 21 Jun 2011 23:33:16 +0000 (-0500)
Subject: dir: Fix DRead
X-Git-Tag: upstream/1.8.0_pre1^2~3611
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=9f8757650c8985785f0934bead2314a2acff21e1;p=packages%2Fo%2Fopenafs.git

dir: Fix DRead

DRead was missing a return statement in one of the cases where we
found the buffer we were looking for, so we locked the buffer but kept
looking. Return it instead.

Change-Id: If72a0ba3ce60a847f2796b51a82f0f473bbc608a
Reviewed-on: http://gerrit.openafs.org/4877
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/dir/buffer.c b/src/dir/buffer.c
index 8a99d4b68..e90334529 100644
--- a/src/dir/buffer.c
+++ b/src/dir/buffer.c
@@ -210,6 +210,7 @@ DRead(dir_file_t fid, int page, struct DirBuffer *entry)
 			ReleaseWriteLock(&tb->lock);
 			entry->buffer = tb;
 			entry->data = tb->data;
+			return 0;
 		    }
 		} else
 		    break;