2017-08-10 16:23 GMT+02:00 Joël Krähemann <jkraehem...@gmail.com>: > Hi James, > > It is common that you use for storing uint8_t an entire word. > The use of a uint8_t pointer confused me as you are pointing > to a struct. > > Bests, > Joël > > > On Thu, Aug 10, 2017 at 4:10 PM, James Cowgill <jcowg...@debian.org> > wrote: > > Hi, > > > > On 10/08/17 08:31, Joël Krähemann wrote: > >> Package: lv2-dev > >> Version: 1.14.0~dfsg1-1 > >> Severity: important > >> > >> Dear Maintainer, > >> > >> The following header makes use of smallest possible pointer in > LV2_Event_Buffer struct's data field. > >> > >> lv2/lv2plug.in/ns/ext/event/event.h > >> > >> Please change it to biggest possible pointer. It should be definitely > void* type because the memory > >> pointed by data shall contain another struct LV2_Event. > >> > >> This describes an integer overflow. There shouldn't be any overflow. > > > > I'm afraid I don't see what the problem is here, or where the integer > > overflow is. The data field is casted to an appropriate pointer type > > whenever it is used and doing that is portable if you're careful. > > Hi Joel,
You still think it is a bug? Did you contacted upstream author about this issue already? best regards mira
