Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 4 Aug 2006 13:29:58 GMT
From:      Roman Divacky <rdivacky@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 103164 for review
Message-ID:  <200608041329.k74DTwj2088629@repoman.freebsd.org>

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

Change 103164 by rdivacky@rdivacky_witten on 2006/08/04 13:29:10

	The futex_put() must be called symmetrically to futex_get() so do it in the FUTEX_WAKE_OP
	case. This fixes the memleak with realplay.

Affected files ...

.. //depot/projects/soc2006/rdivacky_linuxolator/compat/linux/linux_futex.c#15 edit

Differences ...

==== //depot/projects/soc2006/rdivacky_linuxolator/compat/linux/linux_futex.c#15 (text+ko) ====

@@ -252,9 +252,9 @@
 		   	op_ret = 0;
 		   	/* Linux always puts there 0 retries */
    		   	op_ret += futex_wake(f2, 0, NULL);
-			futex_put(f2);
 			ret += op_ret;
 		}
+		futex_put(f2);
 		td->td_retval[0] = ret;
 	   	break;
 



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