From 304f21cb7094e68224b22f8a91144f80207d7a3c Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Thu, 22 Oct 2009 00:31:10 +0100 Subject: [PATCH] Fix fall out from removal of memset casts In places where we're doing pointer arithmetic, we must cast to (char *), because pointer arithmetic on a void * isn't permitted by the C standard. Sadly gcc lets us get away with it 'for convenience'. Reinstate the necessary casts. Reviewed-on: http://gerrit.openafs.org/707 Tested-by: Jeffrey Altman Reviewed-by: Jeffrey Altman --- src/ptserver/ubik.c | 2 +- src/rxkad/ticket5.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ptserver/ubik.c b/src/ptserver/ubik.c index 80c314729..4cec9f681 100644 --- a/src/ptserver/ubik.c +++ b/src/ptserver/ubik.c @@ -160,7 +160,7 @@ ubik_Read(struct ubik_trans *tt, void *buf, afs_int32 len) return (1); } if (status < len) - memset(buf + status, 0, len - status); + memset((char *)buf + status, 0, len - status); return (0); } diff --git a/src/rxkad/ticket5.c b/src/rxkad/ticket5.c index c8e95f49a..da90081f3 100644 --- a/src/rxkad/ticket5.c +++ b/src/rxkad/ticket5.c @@ -480,7 +480,7 @@ krb5_des_decrypt(struct ktc_encryptionKey *key, int etype, void *in, des_cbc_encrypt(in, out, insz, s, &ivec, 0); memcpy(cksum, (char *)out + CONFOUNDERSZ, cksumsz); - memset(out + CONFOUNDERSZ, 0, cksumsz); + memset((char *)out + CONFOUNDERSZ, 0, cksumsz); if (cksum_func) ret = (*cksum_func) (out, insz, cksum, cksumsz, key); -- 2.39.5