[lwip-devel] [bug #50097] Disabling struct packing should be official

[Daniel Elstner]

Follow-up Comment #2, bug #50097 (project lwip):

Hm, one way would be to use static assertions to verify the size of structs in
the proto headers. E.g. have something like this:


#define STATIC_ASSERT(cond, name) typedef char assert_##name[(cond) ? 1 : -1]
struct foo_header { u16 a; u8 b; u8 c; u32 d; };
STATIC_ASSERT(sizeof(struct foo_header) == 8, foo_header_size);


Additionally, if struct packing is disabled there should be a check that
PBUF_LINK_HLEN (which includes ETH_PAD_SIZE) is a multiple of MEM_ALIGNMENT.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?50097>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/