Hi Paolo,<br><br>Shouldn&#39;t the other case that returns NULL in mono_class_create_from_typedef use mono_class_set_failure as well? From now on I&#39;ll send patches using .diff extension.<br><br>Thanks for the review!<br>
Rodrigo<br><br><div><span class="gmail_quote">On 7/9/07, <b class="gmail_sendername">Paolo Molaro</b> &lt;<a href="mailto:lupus@ximian.com">lupus@ximian.com</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On 07/06/07 Rodrigo Kumpera wrote:<br>&gt; The attached patch fixes #82022. The fix is quite simple, it just checks if<br>&gt; the enum field is null and abort the loading process. But there are two<br>&gt; issues with this patch, first it could return a more informative message
<br>&gt; about the problem and I&#39;m not sure if it won&#39;t leak memory or cause future<br>&gt; problems.<br><br>Yes, this will cause a leak. Use mono_class_set_failure() to make<br>sure the runtime considers the type as faulty and return it.
<br><br>BTW, if you name your patch files as name.diff gmail will properly<br>attach them as text files instead of binary blobs making the review<br>easier.<br>Thanks!<br><br>lupus<br><br>--<br>-----------------------------------------------------------------
<br><a href="mailto:lupus@debian.org">lupus@debian.org</a>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; debian/rules<br><a href="mailto:lupus@ximian.com">lupus@ximian.com</a>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Monkeys do it better<br>_______________________________________________
<br>Mono-devel-list mailing list<br><a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br><a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list">http://lists.ximian.com/mailman/listinfo/mono-devel-list
</a><br></blockquote></div><br>