<div dir="ltr"><pre style="margin-top:0px;margin-bottom:0px;padding:0px;font-family:arial,'courier new',courier,ËÎÌå,monospace;white-space:pre-wrap;word-wrap:break-word;color:rgb(51,51,51);font-size:14px;line-height:24px;background-color:rgb(241,254,221)">
I understand. Thanks for your reminding.</pre></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/6 J¨¢n Tomko <span dir="ltr"><<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 09/06/2013 07:35 AM, Hongwei Bi wrote:<br>
> Remove unnecessary 'addrs = NULL' in<br>
> qemuDomainAssignS390Addresses() and<br>
> qemuDomainAssignPCIAddresses() due to<br>
> the related Free function in label cleanup.<br>
<br>
</div>But it is necessary, because we assign the value of addrs to priv->ccwaddrs.<br>
If we didn't set addrs to NULL, it would get freed by that function in the<br>
cleanup: section and priv->ccwaddrs would point to freed memory.<br>
<br>
Jan<br>
</blockquote></div><br></div>