[PATCH 2/9] Fix size capping in write_pieced_value
Yao Qi
qiyaoltc@gmail.com
Thu Apr 13 08:18:00 GMT 2017
Andreas Arnez <arnez@linux.vnet.ibm.com> writes:
> A field in a structure composed of DWARF pieces overlaps one or more of
> those pieces. When writing to the field, the beginning of the first and
> the end of the last of those pieces may have to be skipped. But the
Do you have an example?
> logic in write_pieced_value for handling this is flawed when there are
> actually bits to skip at the beginning of the first piece: it truncates
> the piece size towards the end *before* accounting for the skipped bits
> at the beginning instead of the other way around.
>
> Note that the same bug was already found in read_pieced_value and fixed
> there (but not in write_pieced_value), see PR 15391.
Can we share the code in write_pieced_value and read_pieced_value? The
code computing offsets and bits should be shared. Also, we need more
comments in the code to explain these offsets and bits, a diagram about
the relationships of these bits and offsets is quite helpful.
--
Yao (齐尧)
More information about the Gdb-patches
mailing list