In winbind_rpc.c:query_user_list, the return code of rpccli_samr_QueryDisplayInfo() is not properly evaluated before accessing the disp_info results, which leads to an error if the rpccli command failed. Patch follows next...
btw: this is in the rpc methods
Created attachment 6177 [details] patch with bugfix for 3.5 Fix that applies to v3-5-test. This is the way master also does this, but here the fix was introduced in a refactoring commit c83e8cbe4ae3192b576b47bfae2a4ba90d6e0ce9. I am not 100% sure that this is correct in all cases (e.g., what if NT_STATUS_NO_MORE_ENTRIES is returned? should we pass that on or return NT_STATUS_OK intstead?) But all my tests were ok with the patch.
Comment on attachment 6177 [details] patch with bugfix for 3.5 Looks, good and matches the logic v3-6-test and master
Pushed to v3-5-test. Will be included in the next release. Closing out bug report. Thanks!