[PATCH] Address some -O3 issues on Ubuntu 10.04
Stefan Metzmacher
metze at samba.org
Thu Aug 4 21:31:03 UTC 2016
Hi Michael,
>> diff --git a/source4/dsdb/repl/drepl_out_helpers.c b/source4/dsdb/repl/drepl_out_helpers.c
>> index bf8a372..29bb9d5 100644
>> --- a/source4/dsdb/repl/drepl_out_helpers.c
>> +++ b/source4/dsdb/repl/drepl_out_helpers.c
>> @@ -446,6 +446,9 @@ static void dreplsrv_op_pull_source_get_changes_trigger(struct tevent_req *req)
>> if (!W_ERROR_IS_OK(werr)) {
>> DEBUG(0,(__location__ ": Failed to convert UDV for %s : %s\n",
>> ldb_dn_get_linearized(partition->dn), win_errstr(werr)));
>> + /* The above error can only be OUT_OF_MEMORY. */
>> + tevent_req_nterror(req, NT_STATUS_NO_MEMORY);
>> + return;
>
> Why not use tevent_req_werror(req, werr) ?
We can't mix nterror and werror
tevent_req_ntstatus(req, werror_to_ntstatus(werr));
would work.
Otherwise it looks good to me.
metze
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160804/a4f05dcf/signature.sig>
More information about the samba-technical
mailing list