<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 06/30/2015 12:07 PM, Ludwig Krispenz
wrote:<br>
</div>
<blockquote cite="mid:55926A72.9040205@redhat.com" type="cite">added
verification for issue reported in ticket 5088 and sanity checks
requested in review for patch 0014
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<font face="Times New Roman, Times, serif">Hello,<br>
<br>
The fix looks good except those sanity settings:<br>
</font>
<ul>
<li><font face="Times New Roman, Times, serif">In
ipa_topo_post_del, tsegm needs to be NULL initialized</font></li>
<li><font face="Times New Roman, Times, serif">In
ipa_topo_check_segment_is_valid or ipa_topo_pre_add, I think
*errtxt should be initialized to NULL </font></li>
</ul>
<p><font face="Times New Roman, Times, serif">thanks<br>
thierry<br>
</font></p>
</body>
</html>