We should fail if we didn't get DCERPC_BIND_ACK_RESULT_ACCEPTANCE.
It's also not needed to require a single array element.
We already checked above that we have at least one.
The next patch will all bind time feature negotiation
and that means we'll have 2 array elements...
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
return False;
}
- if (r->num_results != 0x1 || ctx.result != 0) {
- DEBUG(2,("bind_rpc_pipe: bind denied results: %d reason: %x\n",
- r->num_results, ctx.reason.value));
+ if (ctx.result != DCERPC_BIND_ACK_RESULT_ACCEPTANCE) {
+ DBG_NOTICE("bind denied result: %d reason: %x\n",
+ ctx.result, ctx.reason.value);
+ return false;
}
DEBUG(5,("check_bind_response: accepted!\n"));