Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 17 Jan 2007 22:34:17 GMT
From:      Roman Divacky <rdivacky@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 113077 for review
Message-ID:  <200701172234.l0HMYHXj076054@repoman.freebsd.org>

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

Change 113077 by rdivacky@rdivacky_witten on 2007/01/17 22:33:40

	Dont expose em->shared to the outside world before its properly 
	initialized. Might not affect anything but its at least a better
	coding style.

Affected files ...

.. //depot/projects/linuxolator/src/sys/compat/linux/linux_emul.c#21 edit

Differences ...

==== //depot/projects/linuxolator/src/sys/compat/linux/linux_emul.c#21 (text+ko) ====

@@ -91,11 +91,11 @@
 			struct linux_emuldata_shared *s;
 
 			s = malloc(sizeof *s, M_LINUX, M_WAITOK | M_ZERO);
-			em->shared = s;
 			s->refs = 1;
 			s->group_pid = child;
 
 			LIST_INIT(&s->threads);
+			em->shared = s;
 		}
 		p = pfind(child);
 		KASSERT(p != NULL, ("process not found in proc_init\n"));



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