Error building if_fatm.c

From: Craig Rodrigues <rodrigc_at_crodrigues.org>
Date: Tue, 23 Dec 2003 19:50:24 -0500
Hi,

I just cvsup'd current and got the following error when
building if_fatm.c:

cc -c -O -pipe -mcpu=pentiumpro -Wall -Wredundant-decls -Wnested-externs -Wstric
t-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -fforma
t-extensions -std=c99 -g -nostdinc -I-  -I. -I/usr/src/sys -I/usr/src/sys/contri
b/dev/acpica -I/usr/src/sys/contrib/ipfilter -I/usr/src/sys/contrib/dev/ath -I/u
sr/src/sys/contrib/dev/ath/freebsd -I/usr/src/sys/contrib/ngatm -D_KERNEL -inclu
de opt_global.h -fno-common -finline-limit=15000 -fno-strict-aliasing  -mno-alig
n-long-strings -mpreferred-stack-boundary=2 -ffreestanding -Werror  /usr/src/sys
/dev/fatm/if_fatm.c
/usr/src/sys/dev/fatm/if_fatm.c: In function `fatm_open_vcc':
/usr/src/sys/dev/fatm/if_fatm.c:2285: warning: comparison between pointer and integer
*** Error code 1


I think that this is due to the new definition of NULL
in <sys/_null.h>.

Here is the offending code:

   2227         int error;

   2285         if (error != NULL) {
   2286                 sc->vccs[op->param.vci] = NULL;
   2287                 sc->open_vccs--;
   2288                 goto done;
   2289         }


Is the correct fix this?


--- sys/dev/fatm/if_fatm.c.orig	Tue Dec 23 19:46:08 2003
+++ sys/dev/fatm/if_fatm.c	Tue Dec 23 19:46:18 2003
_at__at_ -2282,7 +2282,7 _at__at_
 	sc->open_vccs++;
 
 	error = fatm_load_vc(sc, vc);
-	if (error != NULL) {
+	if (error != 0) {
 		sc->vccs[op->param.vci] = NULL;
 		sc->open_vccs--;
 		goto done;


-- 
Craig Rodrigues        
http://crodrigues.org
rodrigc_at_crodrigues.org
Received on Tue Dec 23 2003 - 15:50:18 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:35 UTC