Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 21 Jan 2006 14:29:31 GMT
From:      Robert Watson <rwatson@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 90088 for review
Message-ID:  <200601211429.k0LETVCR084604@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=90088

Change 90088 by rwatson@rwatson_sesame on 2006/01/21 14:29:29

	Pointer is almost not a boolean.
	
	Submitted by:	phk
	Found with:	FlexeLint

Affected files ...

.. //depot/projects/trustedbsd/openbsm/libbsm/bsm_audit.c#16 edit

Differences ...

==== //depot/projects/trustedbsd/openbsm/libbsm/bsm_audit.c#16 (text+ko) ====

@@ -237,7 +237,7 @@
 	token_t *tok;
 
 	/* Free the token list */
-	while ((tok = TAILQ_FIRST(&rec->token_q))) {
+	while ((tok = TAILQ_FIRST(&rec->token_q)) != NULL) {
 		TAILQ_REMOVE(&rec->token_q, tok, tokens);
 		free(tok->t_data);
 		free(tok);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200601211429.k0LETVCR084604>