<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 30, 2014 at 5:09 PM, Greg KH <span dir="ltr"><<a href="mailto:gregkh@linuxfoundation.org" target="_blank">gregkh@linuxfoundation.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>> +/* make sure the hostname is not "localhost" */<br>
> +static bool is_localhost(const char *hostname) {<br>
> +        char *hostend;<br>
> +<br>
> +        assert(hostname);<br>
<br>
</div></div>You only call this in one place, with an array on the stack, so it can<br>
never be NULL.  So why check this at all?<br>
<br>
Other than that minor nit, this looks good, thanks for adding it.<br></blockquote><div><br></div><div>I was thinking that this function might be called in the future in other places as well so</div><div>it doesn't hurt to check. But I can nuke the assert.</div>
<div><br></div><div>-Eugene</div></div></div></div>