viced-malloc-dont-make-assumptions-about-the-compiler-20020524
authorDerrick Brashear <shadow@dementia.org>
Fri, 24 May 2002 23:05:47 +0000 (23:05 +0000)
committerDerrick Brashear <shadow@dementia.org>
Fri, 24 May 2002 23:05:47 +0000 (23:05 +0000)
commitbbab90943637385742923c8bebebdd750a283e5c
tree0ca43ce42417cb3a053ba648257a0bc3fd9ba8cc
parent155c13192339239dd1d794210b5bcedfb3feea15
viced-malloc-dont-make-assumptions-about-the-compiler-20020524

based on report from lha@stacken.kth.se
"identP = (struct Identity *)malloc(1);

This can't be right, there should not be an assumption how the
c-compiler will lay out code. I find it acceptable to do something like this

   /* sizeof valid + offset of valid. */
   i.. = malloc(sizeof(identP->valid) + (int)&(((struct Identity *)0)->valid));

But then, why not just allocate a whole identP and be happy. A afsUUID
is not that bloted (16 bytes) so I can't see why way this is
chosen. 20 bytes will probably be allocated on 32 bit platforms for
the whole struct Identity.

"
src/viced/host.c