Rémi Bernon (@rbernon) commented about programs/explorer/systray.c:
/* FIXME: if statement only needed because we don't support interprocess * icon handles */
- if ((nid.uFlags & NIF_ICON) && cds->cbData > sizeof(*data))
- if ((nid.uFlags & NIF_ICON) && cds->cbData > ((char*)icon_data - (char*)data))
You can keep the sizeof I think?