Date: Sat, 21 Jan 2006 14:08:03 GMT From: Robert Watson <rwatson@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 90083 for review Message-ID: <200601211408.k0LE83H1083226@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=90083 Change 90083 by rwatson@rwatson_sesame on 2006/01/21 14:07:04 Add an assert to help FlexeLint figure out GET_TOKEN_AREA(). phk and I can't see the bug in the macro that causes FlexeLint to get unhappy, which doesn't mean it doesn't exist, just that it's got us stumped, so the assert helps FlexeLint figure out what we meant (and maybe said). Affected files ... .. //depot/projects/trustedbsd/openbsm/libbsm/bsm_token.c#29 edit Differences ... ==== //depot/projects/trustedbsd/openbsm/libbsm/bsm_token.c#29 (text+ko) ==== @@ -47,6 +47,7 @@ #include <netinet/in_systm.h> #include <netinet/ip.h> +#include <assert.h> #include <errno.h> #include <string.h> #include <stdlib.h> @@ -67,6 +68,7 @@ } else \ memset(dptr, 0, (length)); \ } \ + assert(t == NULL || dptr != NULL); \ } while (0) /*
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200601211408.k0LE83H1083226>