<div dir="ltr">Majnemer, I don't follow.  If Data.size() > 0 and Buffer.size() == 0, then the old code would fail the first check "Data.size() < Buffer.size()" and the second check "Offset > Buffer.size() - Data.size()"  (this would actually overflow since Buffer.size() would be 0).<div><br></div><div>So then it would memcpy with a size of 0 and return Error::success().</div><div><br></div><div>Where are you seeing that it returns insufficient_buffer?</div><div><br></div><div>Now it just immediately returns success if you're attempting to write nothing, which seems right to me.</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 14, 2016 at 2:27 PM David Majnemer <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">majnemer added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Did this get an LGTM? This change doesn't look right. The old code would return `msf_error_code::insufficient_buffer` if `Data.size()` was greater than zero.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26638" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26638</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>