You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by sz...@apache.org on 2008/12/04 23:32:38 UTC

svn commit: r723472 - in /hadoop/core/branches/branch-0.19: CHANGES.txt src/contrib/fuse-dfs/src/fuse_dfs.c

Author: szetszwo
Date: Thu Dec  4 14:32:38 2008
New Revision: 723472

URL: http://svn.apache.org/viewvc?rev=723472&view=rev
Log:
HADOOP-4727. Fix a group checking bug in fill_stat_structure(...) in fuse-dfs.  (Brian Bockelman via szetszwo)

Modified:
    hadoop/core/branches/branch-0.19/CHANGES.txt
    hadoop/core/branches/branch-0.19/src/contrib/fuse-dfs/src/fuse_dfs.c

Modified: hadoop/core/branches/branch-0.19/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.19/CHANGES.txt?rev=723472&r1=723471&r2=723472&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.19/CHANGES.txt (original)
+++ hadoop/core/branches/branch-0.19/CHANGES.txt Thu Dec  4 14:32:38 2008
@@ -21,6 +21,9 @@
     HADOOP-4508. Fix FSDataOutputStream.getPos() for append. (dhruba via
     szetszwo)
 
+    HADOOP-4727. Fix a group checking bug in fill_stat_structure(...) in
+    fuse-dfs.  (Brian Bockelman via szetszwo)
+
 Release 0.19.0 - 2008-11-18
 
   INCOMPATIBLE CHANGES

Modified: hadoop/core/branches/branch-0.19/src/contrib/fuse-dfs/src/fuse_dfs.c
URL: http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.19/src/contrib/fuse-dfs/src/fuse_dfs.c?rev=723472&r1=723471&r2=723472&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.19/src/contrib/fuse-dfs/src/fuse_dfs.c (original)
+++ hadoop/core/branches/branch-0.19/src/contrib/fuse-dfs/src/fuse_dfs.c Thu Dec  4 14:32:38 2008
@@ -407,7 +407,7 @@
 #endif
   gid_t group_id = default_id;
 #if PERMS
-  if (info->mGroup == NULL) {
+  if (info->mGroup != NULL) {
     //
     // Critical section - protect from concurrent calls in different threads since
     // the struct below is static.