<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 14, 2014 at 1:30 PM, Marc-André Lureau <span dir="ltr"><<a href="mailto:mlureau@redhat.com" target="_blank">mlureau@redhat.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><br>
<br>
----- Original Message -----<br>
> Hi Marc,<br>
><br>
><br>
> On Mon, Jul 14, 2014 at 12:19 PM, Marc-André Lureau < <a href="mailto:mlureau@redhat.com" target="_blank">mlureau@redhat.com</a> ><br>
> wrote:<br>
><br>
><br>
> Hi,<br>
><br>
> All of those changes are OOM conditions. How can we shut coverity on those?<br>
><br>
> Yes, there is a simple way and that's what we have in Evolution Data Server.<br>
> Long story short, we can just add some comments in the code like: /*<br>
> coverity[unchecked_value] */<br>
> Please, take a look on:<br>
> <a href="https://git.gnome.org/browse/evolution-data-server/commit/?id=a317be96dcffe6ce2c5900fe4e76d955e2294ce9" target="_blank">https://git.gnome.org/browse/evolution-data-server/commit/?id=a317be96dcffe6ce2c5900fe4e76d955e2294ce9</a><br>



<br>
</div>I don't see allocation failure workarounds here.<br></blockquote><div><br></div><div>The workaround would be adding something like this:</div><div><br></div><div>/* We don't handle OOM conditions in spice-gtk */</div>

<div>
/* coverity[null_returns] */</div><div> <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<div>
> Can I go for it?<br>
<br>
</div>For consistency, I would prefer to avoid checking for allocation failures.<br>
<br>
Also using glib API for allocations for the same reason.<br>
<div><br>
><br>
> Why does it warn here and not for other allocations?<br>
><br>
> Hmmm. I don't know, maybe these are the cases where we are using the members<br>
> of the structure right after the allocation fail, but it's just a guess.<br>
><br>
<br>
</div>Could malloc & free do not have annotations like the glib ones.<br>
<br>
What happens for example if you replace malloc() by g_malloc(), and free() by g_free() in the function below?<br></blockquote><div><br></div><div>I don't think that would help as one of the Coverity's complaints came from: "data = g_malloc(d->area.width * d->area.height * 3);"</div>


<div><br></div></div><br clear="all"><div>Best Regards,</div>-- <br><div>Fabiano Fidêncio</div>
</div></div>