[src/trunk]: src/sys/kern Only kmem_free() the filename if we have one.

[elad <elad%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/4de1ccaf8f4e
branches:  trunk
changeset: 750331:4de1ccaf8f4e
user:      elad <elad%NetBSD.org@localhost>
date:      Fri Dec 25 22:57:54 2009 +0000

description:
Only kmem_free() the filename if we have one.

diffstat:

 sys/kern/kern_verifiedexec.c |  9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diffs (32 lines):

diff -r 5dd816701911 -r 4de1ccaf8f4e sys/kern/kern_verifiedexec.c
--- a/sys/kern/kern_verifiedexec.c      Fri Dec 25 20:07:18 2009 +0000
+++ b/sys/kern/kern_verifiedexec.c      Fri Dec 25 22:57:54 2009 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: kern_verifiedexec.c,v 1.118 2009/11/07 07:27:49 cegger Exp $   */
+/*     $NetBSD: kern_verifiedexec.c,v 1.119 2009/12/25 22:57:54 elad Exp $     */
 
 /*-
  * Copyright (c) 2005, 2006 Elad Efrat <elad%NetBSD.org@localhost>
@@ -29,7 +29,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_verifiedexec.c,v 1.118 2009/11/07 07:27:49 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_verifiedexec.c,v 1.119 2009/12/25 22:57:54 elad Exp $");
 
 #include "opt_veriexec.h"
 
@@ -913,9 +913,12 @@
                         * entries so we can destroy the object.
                         */
 
-                       kmem_free(vfe->filename, vfe->filename_len);
+                       if (vfe->filename_len > 0)
+                               kmem_free(vfe->filename, vfe->filename_len);
+
                        vfe->filename = NULL;
                        vfe->filename_len = 0;
+
                        rw_downgrade(&veriexec_op_lock);
                }