PERFORCE change 75890 for review

[Robert Watson]

http://perforce.freebsd.org/chv.cgi?CH=75890

Change 75890 by rwatson at rwatson_paprika on 2005/04/24 16:20:44

	Answer the question in the comment as to whether au_free_token()
	should be a macro -- answer is no, because it's a defined part of
	the API/ABI as a symbol, and in order to keep the contents of the
	token structure out of the ABI, it has to be a function.
	
	Minor reformatting.

Affected files ...

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

Differences ...

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

@@ -55,17 +55,16 @@
 
 /* 
  * This call frees a token_t and its internal data.  
- *
- * XXX  Should it be a macro for speed?  
  */
-void au_free_token(token_t *tok)
+void
+au_free_token(token_t *tok)
 {
-    if (tok)
-    {
+
+	if (tok != NULL) {
 		if (tok->t_data)
 			free(tok->t_data);
 		free(tok);
-    }
+	}
 }
 
 /*
To Unsubscribe: send mail to majordomo at trustedbsd.org
with "unsubscribe trustedbsd-cvs" in the body of the message