<div dir="ltr">Hi,<div><br></div><div>A while ago there was this commit:<br><a href="https://freenginx.org/hg/nginx/rev/d596a1fb8b9c">https://freenginx.org/hg/nginx/rev/d596a1fb8b9c</a><br>"Cleaned up unsafe ngx_errno and ngx_socket_errno usage."<br><br>I just noticed that one part of the above fix was incomplete. In src/os/unix/ngx_freebsd_init.c (tip) starting at line 125 we have:<br><br>    if (sysctlbyname("kern.osrelease",<br>                     ngx_freebsd_kern_osrelease, &size, NULL, 0) == -1) {<br>        err = ngx_errno;<br><br>        ngx_log_error(NGX_LOG_ALERT, log, err,<br>                      "sysctlbyname(kern.osrelease) failed");<br><br>        if (ngx_errno != NGX_ENOMEM) {<br>            return NGX_ERROR;<br>        }<br><br>In the 'if' condition at the end, I think 'ngx_errno' should instead be 'err' (similar to the code block just previous). Sorry, I didn't create a patch, but the fix is simple.</div><div><br></div><div>Regards,</div><div>David</div></div>