[PATCH] winbindd: disconnect child process if request is cancelled at main process

Stefan (metze) Metzmacher metze at samba.org
Wed Jun 24 04:43:30 MDT 2015


Hi Uri,

> Thanks very much for reviewing.
> I've tried following your suggestions and the example and indeed the
> result is much more pleasant. Review appreciated.
> 
> One change I did make was to use the subrequest and not the child
> pointer as a test for whether we have anything to do, and this is
> because unlike the smbcli example, the request may be queued initially
> with no real "io" request. In that state we want the cleanup to be a
> no-op. More generally it's the presence of a subrequest that calls for
> cleanup considerations and if there isn't one, then the cleanup should
> be a no-op. Given that, I did not include the setting of child to NULL
> once we're done with it, although one might want that out of general
> hygiene considerations.

I guess the /* we're done and don't need a reference to the child anymore */
comment is confusing without setting state->child = NULL;

Please create a bug report and add a reference to the commit message
and add a comment before ret = wb_simple_trans_recv(subreq, state,
&state->response, &err);
explaining that we don't call TALLOC_FREE(subreq); and defer that to
wb_child_request_cleanup().

Otherwise Reviewed-by: me

Thanks!
metze

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20150624/77c03c66/attachment.pgp>


More information about the samba-technical mailing list