[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