<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 15, 2016 at 3:23 PM, Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
--- a/clients/ivi-shell-user-interface.c<br>
+++ b/clients/ivi-shell-user-interface.c<br>
@@ -40,6 +40,7 @@<br>
 #include "shared/config-parser.h"<br>
 #include "shared/helpers.h"<br>
 #include "shared/os-compatibility.h"<br>
+#include "shared/xalloc.h"<br>
 #include "shared/zalloc.h"<br>
 #include "ivi-application-client-protocol.h"<br>
 #include "ivi-hmi-controller-client-protocol.h"<br>
@@ -164,18 +165,6 @@ hmi_homescreen_setting {<br>
 };<br>
<br>
 static void *<br>
-fail_on_null(void *p, size_t size, char *file, int32_t line)<br>
-{<br>
-       if (size && !p) {<br>
-               fprintf(stderr, "%s(%d) %zd: out of memory\n",<br>
-                       file, line, size);<br>
-               exit(EXIT_FAILURE);<br>
-       }<br>
-<br>
-       return p;<br>
-}<br></blockquote><div><br></div><div>The deleted ones in ivi purposely does not fail if size==0. But it appears they are only called by the MEM_ALLOC macro and that is only called with sizeof() which is not going to be zero, so this is ok.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
-               fail_on_null(window->xdg_surface);<br>
+               fail_on_null(window->xdg_surface, 1, __FILE__, __LINE__);<br></blockquote><div><br></div><div>I think you either want to use sizeof(*window->xdg_surface) or 0 for the size.</div><div><br></div></div><br></div></div>