>>! In D4527#33, @samsonov wrote:
>>> Why not internal_strncpy(descr->name, vars[i].name_pos, 
>>> Min(descr->name_size - 1, vars[i].name_len)) ?
>> Because that would not write \0 after the string (we're copying just a part 
>> of the string pointed to by vars[i].name_pos). So I would have to memset the 
>> whole buffer to zero before. Should I do that, instead of the strncat? (The 
>> strncat is also used in asan_report.cc:436).
> Right, let's keep strncat here then.

Ok. So, are you fine with how the patch looks like now?

http://reviews.llvm.org/D4527



_______________________________________________
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to