[email protected]
[Top] [All Lists]

PERFORCE change 114195 for review

Subject: PERFORCE change 114195 for review
From: Roman Divacky
Date: Wed, 7 Feb 2007 18:33:37 GMT
http://perforce.freebsd.org/chv.cgi?CH=114195

Change 114195 by [email protected]_witten on 2007/02/07 18:33:12

        Cache the em->pdeath_signal in a local variable and move the copyout
        out of the emul_lock coverage.

Affected files ...

.. //depot/projects/linuxolator/src/sys/compat/linux/linux_misc.c#56 edit

Differences ...

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

@@ -1652,6 +1652,7 @@
        struct proc *p = td->td_proc;
        char comm[LINUX_MAX_COMM_LEN];
        struct linux_emuldata *em;
+       int pdeath_signal;
 
 #ifdef DEBUG
        if (ldebug(prctl))
@@ -1671,10 +1672,11 @@
        case LINUX_PR_GET_PDEATHSIG:
                em = em_find(p, EMUL_DOLOCK);
                KASSERT(em != NULL, ("prctl: emuldata not found.\n"));
-               error = copyout(&em->pdeath_signal,
+               pdeath_signal = em->pdeath_signal;
+               EMUL_UNLOCK(&emul_lock);
+               error = copyout(&pdeath_signal,
                    (void *)(register_t)args->arg2,
-                   sizeof(em->pdeath_signal));
-               EMUL_UNLOCK(&emul_lock);
+                   sizeof(pdeath_signal));
                break;
        case LINUX_PR_SET_NAME:
                /*
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/p4-projects
To unsubscribe, send any mail to "[email protected]"

<Prev in Thread] Current Thread [Next in Thread>
  • PERFORCE change 114195 for review, Roman Divacky <=