winbind: dom_sid_parse_endp always initializes "endp" when ok
authorVolker Lendecke <vl@samba.org>
Thu, 1 Dec 2016 16:16:14 +0000 (16:16 +0000)
committerJeremy Allison <jra@samba.org>
Tue, 6 Dec 2016 23:11:03 +0000 (00:11 +0100)
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Dec  7 00:11:03 CET 2016 on sn-devel-144

source3/winbindd/winbindd_util.c

index a35b59e5fb8bad446606798b0508ffd560a95ed6..38e4b8b8edb244e6235429db126c36c435088bf2 100644 (file)
@@ -1651,7 +1651,7 @@ bool parse_sidlist(TALLOC_CTX *mem_ctx, const char *sidstr,
                        DEBUG(1, ("Could not parse sid %s\n", p));
                        return false;
                }
-               if ((q == NULL) || (q[0] != '\n')) {
+               if (q[0] != '\n') {
                        DEBUG(1, ("Got invalid sidstr: %s\n", p));
                        return false;
                }