]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
LINUX: Use correct type of error in flock code
authorDerrick Brashear <shadow@dementia.org>
Thu, 6 Jan 2011 06:37:44 +0000 (01:37 -0500)
committerRuss Allbery <rra@debian.org>
Fri, 7 Jan 2011 04:10:49 +0000 (20:10 -0800)
The flock code gets a kernel error code from afs_posix_lock_file()
but then passes this error code through afs_convert_code (which sees
that it is already negative, and returns EIO). Instead, we should just
return afs_posix_lock_file()'s code direct to our caller.

Reviewed-on: http://gerrit.openafs.org/2534
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit 5abe706a7e237d3872db3638d947a14c935370d5)
Change-Id: I8e12c9804cc81c8c88f820f55abd4ce863525c9a
Reviewed-on: http://gerrit.openafs.org/3533

src/afs/LINUX/osi_vnodeops.c

index ed4354ee2313f82eb3cf5377b46d76597a9183b2..74e17896c75922f7f0bada00f719ebf97c02bc4e 100644 (file)
@@ -493,7 +493,7 @@ afs_linux_lock(struct file *fp, int cmd, struct file_lock *flp)
 #endif /* F_GETLK64 && F_GETLK != F_GETLK64 */
 
     AFS_GLOCK();
-    code = afs_lockctl(vcp, &flock, cmd, credp);
+    code = afs_convert_code(afs_lockctl(vcp, &flock, cmd, credp));
     AFS_GUNLOCK();
 
 #ifdef AFS_LINUX24_ENV
@@ -522,7 +522,7 @@ afs_linux_lock(struct file *fp, int cmd, struct file_lock *flp)
     flp->fl_end = flock.l_start + flock.l_len;
 
     crfree(credp);
-    return afs_convert_code(code);
+    return code;
 
 }