<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 05/13/2013 05:22 PM, Petr Spacek
wrote:<br>
</div>
<blockquote cite="mid:51910527.4080606@redhat.com" type="cite">On
13.5.2013 16:49, Simo Sorce wrote:
<br>
<blockquote type="cite">On Mon, 2013-05-13 at 16:32 +0200, Petr
Spacek wrote:
<br>
<blockquote type="cite">+ if ((ip = inet_addr(ip_str)) ==
INADDR_NONE) {
<br>
</blockquote>
<br>
<br>
This kind of construct is hard to read and debug in gdb
<br>
I would suggest you do:
<br>
ip = inet_addr(ip_str);
<br>
if (ip == INADDER_NONE) {
<br>
</blockquote>
<br>
I agree, done.
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Freeipa-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre>
</blockquote>
I tested the patch, works fine. ACK<br>
<br>
Tomas<br>
</body>
</html>