From 6358c55bacb813395a4612b9f3972bf41a3daed2 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Wed, 22 Sep 2010 23:08:03 -0400 Subject: libfec: fix up pointer warnings in fec magic computation The current fec code casts a pointer to an int which causes warnings on 64bit systems. So create a macro for the duplicate/complicated magic computation, and add an unsigned long cast in it to fix the original problem. Signed-off-by: Mike Frysinger Signed-off-by: Artem Bityutskiy --- lib/libfec.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lib/libfec.c b/lib/libfec.c index 924701f..b19ed6e 100644 --- a/lib/libfec.c +++ b/lib/libfec.c @@ -629,12 +629,13 @@ struct fec_parms { int k, n ; /* parameters of the code */ gf *enc_matrix ; } ; +#define COMP_FEC_MAGIC(fec) \ + (((FEC_MAGIC ^ (fec)->k) ^ (fec)->n) ^ (unsigned long)((fec)->enc_matrix)) void fec_free(struct fec_parms *p) { - if (p==NULL || - p->magic != ( ( (FEC_MAGIC ^ p->k) ^ p->n) ^ (int)(p->enc_matrix)) ) { + if (p==NULL || p->magic != COMP_FEC_MAGIC(p)) { fprintf(stderr, "bad parameters to fec_free\n"); return ; } @@ -666,7 +667,7 @@ fec_new(int k, int n) retval->k = k ; retval->n = n ; retval->enc_matrix = NEW_GF_MATRIX(n, k); - retval->magic = ( ( FEC_MAGIC ^ k) ^ n) ^ (int)(retval->enc_matrix) ; + retval->magic = COMP_FEC_MAGIC(retval); tmp_m = NEW_GF_MATRIX(n, k); /* * fill the matrix with powers of field elements, starting from 0. -- cgit v1.2.3