<div dir="ltr">Hi,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On 26 November 2014 at 22:49, 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"><span class="">On Wed, Nov 26, 2014 at 03:33:16PM +0100, Marek Chalupa wrote:<br>
> On 21 November 2014 at 07:21, Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>><br>
> wrote:<br>
><br>
> > Signed-off-by: Bryce Harrington <<a href="mailto:bryce@osg.samsung.com">bryce@osg.samsung.com</a>><br>
> ><br>
</span><span class="">> > -       if (do_yflip)<br>
> > -               recorder->tmpbuf = NULL;<br>
> > -       else<br>
> > +       if (!do_yflip)<br>
> >                 recorder->tmpbuf = malloc(size);<br>
> ><br>
><br>
> Here you don't check the malloc's result.<br>
<br>
</span>Good point.  The original code didn't either, but<br>
weston_recorder_frame_notify() looks like it assumes tmpbuf is valid and<br>
would crash there otherwise.  I can add this.<br></blockquote><div><br></div><div>I found out that I have add this check when I was testing your patches, so I sent it to list.<br>I hope you don't mind that I took you the 'work' here :)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> Also in weston_recorder_destroy<br>
> there's no check that tmpbuf was allocated before freeing that memory.<br>
<br>
</span>Actually it's unnecessary to check for NULL before calling free().<br>
free(NULL) is a no-op as per `man 3 malloc`.<br>
<br>
Thanks for the review!<br>
<span class="HOEnZb"><font color="#888888"><br>
Bryce<br>
<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Thanks,<br>Marek<br></div></div></div>