s4-libnet Always return after composite_error()

This can and does cause crashes as multiple conflicting sets of callbacks run on
memory that may have been destroyed.

Andrew Bartlett

source4/libnet/groupinfo.c

@@ -93,6 +93,7 @@ static void continue_groupinfo_lookup(struct tevent_req *subreq)
 	   - we're looking for only one at the moment */
 	if (s->lookup.out.rids->count == 0) {
 		composite_error(c, NT_STATUS_NO_SUCH_USER);
+		return;
 	}
 
 	/* TODO: find proper status code for more than one rid found */

source4/libnet/libnet_domain.c

@@ -1137,6 +1137,7 @@ static void continue_samr_close_handle(struct tevent_req *subreq)
 	/* did everything go fine ? */
 	if (!NT_STATUS_IS_OK(s->samrclose.out.result)) {
 		composite_error(c, s->samrclose.out.result);
+		return;
 	}
 
 	composite_done(c);

source4/libnet/userinfo.c

@@ -92,6 +92,7 @@ static void continue_userinfo_lookup(struct tevent_req *subreq)
 	   - we're looking for only one at the moment */
 	if (s->lookup.out.rids->count == 0) {
 		composite_error(c, NT_STATUS_NO_SUCH_USER);
+		return;
 	}
 
 	/* TODO: find proper status code for more than one rid found */