r6057: Don't put the assert in the wrong place :-).

Jeremy.

diff --git a/source/smbd/posix_acls.c b/source/smbd/posix_acls.c
index 33db14aec3a49b04de1f010360e59bb8641b0877..0b95a03dcb2d0ce97d15ba1cbf3133977f0a864c 100644 (file)
--- a/source/smbd/posix_acls.c
+++ b/source/smbd/posix_acls.c
@@ -3888,12 +3888,11 @@ match on group %u -> can write.\n", fname, (unsigned int)*pgid ));
                }
        }
 
-
-  check_stat:
-
        /* If we get here we know ret == 0. */
        SMB_ASSERT(ret == 0);
 
+  check_stat:
+
        for (i = 0; i < current_user.ngroups; i++) {
                if (current_user.groups[i] == psbuf->st_gid) {
                        ret = (psbuf->st_mode & S_IWGRP) ? 1 : 0;
@@ -3904,7 +3903,7 @@ match on owning group %u -> %s.\n", fname, (unsigned int)psbuf->st_gid, ret ? "c
        }
 
        if (i == current_user.ngroups) {
-               SMB_ASSERT(ret == 0);
+               SMB_ASSERT(ret != 1);
                DEBUG(10,("check_posix_acl_group_write: file %s \
 failed to match on user or group in token.\n", fname ));
        }