On 12/10/2021 10:18 PM, Alexandre Oliva wrote:
On Dec 10, 2021, Jeff Law <jeffreya...@gmail.com> wrote:

The patch is clearly safe.  My question is should we have caught this
earlier in the call chain?
Callers will call try_store_by_multiple_pieces if set_storage_via_setmem
fails.  setmem doesn't necessarily need min and max len to do its job,
so if we were to modify callers, it would be just guarding the calls of
try_store_by_multiple_pieces with max_len >= min_len: 3 callers in 2
files, which didn't seem appealing to me.
Thanks for the additional info.  OK for the trunk.

Jeff

Reply via email to