PERFORCE change 100890 for review

John Baldwin jhb at FreeBSD.org
Fri Jul 7 16:01:56 UTC 2006


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

Change 100890 by jhb at jhb_mutex on 2006/07/07 16:01:44

	Don't try to free name in the !uap->name case as it is just
	garbage.

Affected files ...

.. //depot/projects/smpng/sys/kern/uipc_syscalls.c#81 edit

Differences ...

==== //depot/projects/smpng/sys/kern/uipc_syscalls.c#81 (text+ko) ====

@@ -328,8 +328,8 @@
 		if (error == 0)
 			error = copyout(&namelen, uap->anamelen,
 			    sizeof(namelen));
+		free(name, M_SONAME);
 	}
-	free(name, M_SONAME);
 	return (error);
 }
 
@@ -468,7 +468,8 @@
 		sa = NULL;
 	}
 noconnection:
-	free(sa, M_SONAME);
+	if (sa)
+		FREE(sa, M_SONAME);
 
 	/*
 	 * close the new descriptor, assuming someone hasn't ripped it


More information about the p4-projects mailing list