Browse Source

Simplify program flow: eliminate unnecessary continue
Reviewed by: rmeggins
(cherry picked from commit 0a3b950695037a4b9348d60edf1a7863be2e76b7)

Ken Rossato 13 years ago
parent
commit
77eb7602e9
1 changed files with 2 additions and 4 deletions
  1. 2 4
      ldap/servers/plugins/posix-winsync/posix-group-func.c

+ 2 - 4
ldap/servers/plugins/posix-winsync/posix-group-func.c

@@ -479,11 +479,9 @@ addGroupMembership(Slapi_Entry *entry, Slapi_Entry *ad_entry)
         } else {
             v = slapi_value_new_string(uid);
             slapi_ch_free_string(&uid);
-            if (slapi_attr_value_find(muid_attr, slapi_value_get_berval(v)) == 0) {
-                slapi_value_free(&v);
-                continue;
+            if (slapi_attr_value_find(muid_attr, slapi_value_get_berval(v)) != 0) {
+                slapi_valueset_add_value(newvs, v);
             }
-            slapi_valueset_add_value(newvs, v);
             slapi_value_free(&v);
         }
     }