On Thu, 2017-03-30 at 09:56 +0200, Sumit Bose wrote:
On Wed, Mar 29, 2017 at 06:54:09PM +0000, Joakim Tjernlund wrote:
with that BT I looked at the src and it became obvious, the check: https://cgit.freedesktop.org/realmd/adcli/commit/?id=213116ea8a16a10f4def750... needs to move into the else part holding _adcli_info ("Received NetLogon info from: %s", disco->host_name); as disco ptr can be NULL otherwise
d'oh, thank you very much for your analysis. You are right either the check can be move in the else block or disco should be checked for NULL before dereferencing it.
Would you like to send a patch for this? You can either use bugzilla.redhat.com or upstream's bugs.freedesktop.org with product realmd component adcli to open a ticket and attache a patch.
https://bugs.freedesktop.org/show_bug.cgi?id=100466
Jocke