<div dir="ltr">Thanks, i'll adjust my code and send the patch after my build success.</div><div class="gmail_extra"><br><div class="gmail_quote">2016-08-06 1:30 GMT-04:00 Keith Packard <span dir="ltr"><<a href="mailto:keithp@keithp.com" target="_blank">keithp@keithp.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Kayo Hamid <<a href="mailto:khflab@gmail.com">khflab@gmail.com</a>> writes:<br>
<br>
> I'm trying to do one janitor task looking for bad allocations. Someone<br>
> could review my diff and see if is okay?<br>
><br>
> Thanks.<br>
><br>
> --<br>
> Kayo Hamid<br>
</div></div>> diff --git a/dix/enterleave.c b/dix/enterleave.c<br>
> index 1b341f2..8cec9a2 100644<br>
> --- a/dix/enterleave.c<br>
> +++ b/dix/enterleave.c<br>
> @@ -715,6 +715,13 @@ DeliverStateNotifyEvent(<wbr>DeviceIntPtr dev, WindowPtr win)<br>
>      }<br>
><br>
>      sev = ev = xallocarray(evcount, sizeof(xEvent));<br>
> +<br>
> +    if(!ev) {<br>
> +        free(ev);<br>
> +        free(sev);<br>
> +        return BadAlloc;<br>
> +    }<br>
> +<br>
<br>
Good catch on the allocation failure, but in this case you don't need to<br>
free anything, and you should make sure your code compiles without<br>
warnings as this function is void, and so wouldn't need a return value.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
-keith<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Kayo Hamid<br></div></div>
</div>