<div dir="ltr"><div>The <font face="monospace, monospace">MallocAllocator</font> <font face="monospace, monospace">Allocate</font> function is defined as:</div><div><br></div><div><pre style="font-family:Consolas;color:gainsboro;background:rgb(30,30,30)">  <span style="color:rgb(189,99,197)">LLVM_ATTRIBUTE_RETURNS_NONNULL</span> <span style="color:rgb(86,156,214)">void</span> <span style="color:rgb(180,180,180)">*</span><span style="color:rgb(200,200,200)">Allocate</span><span style="color:rgb(180,180,180)">(</span><span style="color:rgb(78,201,176)">size_t</span> <span style="color:rgb(127,127,127)">Size</span><span style="color:rgb(180,180,180)">,</span>
                                                <span style="color:rgb(78,201,176)">size_t</span> <span style="color:rgb(87,166,74)">/*Alignment*/</span><span style="color:rgb(180,180,180)">)</span> <span style="color:rgb(180,180,180)">{</span>
    <span style="color:rgb(86,156,214)">return</span> <span style="color:rgb(200,200,200)">malloc</span><span style="color:rgb(180,180,180)">(</span><span style="color:rgb(127,127,127)">Size</span><span style="color:rgb(180,180,180)">);</span>
  <span style="color:rgb(180,180,180)">}</span>
</pre></div><div>...but <font face="monospace, monospace">malloc</font> can totally return <font face="monospace, monospace">NULL</font>? What's up?</div><div><br></div><div>I assume this is intended to be an optimization/warning silencer? (i.e. LLVM just normally doesn't handle OOM) ...but this can cause security issues (via <font face="monospace, monospace">nullptr</font> deref)?</div><div><br></div><div>On a related note, maybe <font face="monospace, monospace">Allocate</font> should be marked as <font face="monospace, monospace">LLVM_ATTRIBUTE_RETURNS_NOALIAS</font>?</div><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="font-size:12.8px">Sincerely,</span><br style="font-size:12.8px"><span style="font-size:12.8px">Alexander Riccio</span><br style="font-size:12.8px"><span style="font-size:12.8px">--</span><br style="font-size:12.8px"><span style="font-size:12.8px">"Change the world or go home."</span><div style="font-size:12.8px"><a href="http://about.me/ariccio" target="_blank">about.me/ariccio</a></div><div style="font-size:12.8px"><a href="http://about.me/ariccio" target="_blank"><br></a></div><div style="font-size:12.8px">If left to my own devices, I will build more.</div><div style="font-size:12.8px">⁂</div></div></div></div></div></div>
</div>