OpenAFS CVS Commit: openafs/src/viced by shadow

[cvs@GRAND.CENTRAL.ORG]

Update of /cvs/openafs/src/viced
In directory GRAND.CENTRAL.ORG:/data/sb/openafs-stable-1.2/openafs/src/viced

Modified Files:
      Tag: openafs-stable-1_2_x
	host.c 
Log Message:
DELTA STABLE12-viced-malloc-dont-make-assumptions-about-the-compiler-20020524
AUTHOR shadow@dementia.org

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."


--- DELTA config for openafs-stable-1_2_x follows ---
STABLE12-viced-malloc-dont-make-assumptions-about-the-compiler-20020524 openafs/src/viced/host.c 1.7.2.5 1.7.2.6