[src/trunk]: src/sys/dev In the n == VAUDIOCHANS case we need to call mutex_e...

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

details:   https://anonhg.NetBSD.org/src/rev/1e16e49449bc
branches:  trunk
changeset: 349601:1e16e49449bc
user:      pgoyette <pgoyette%NetBSD.org@localhost>
date:      Thu Dec 15 22:01:57 2016 +0000

description:
In the n == VAUDIOCHANS case we need to call mutex_exit() before
returning.


Perhaps this will address PR kern/51721

diffstat:

 sys/dev/audio.c |  8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diffs (30 lines):

diff -r c70fca3ed981 -r 1e16e49449bc sys/dev/audio.c
--- a/sys/dev/audio.c   Thu Dec 15 20:04:36 2016 +0000
+++ b/sys/dev/audio.c   Thu Dec 15 22:01:57 2016 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: audio.c,v 1.278 2016/12/13 17:12:51 christos Exp $     */
+/*     $NetBSD: audio.c,v 1.279 2016/12/15 22:01:57 pgoyette Exp $     */
 
 /*-
  * Copyright (c) 2016 Nathanial Sloss <nathanialsloss%yahoo.com.au@localhost>
@@ -148,7 +148,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: audio.c,v 1.278 2016/12/13 17:12:51 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: audio.c,v 1.279 2016/12/15 22:01:57 pgoyette Exp $");
 
 #include "audio.h"
 #if NAUDIO > 0
@@ -5001,8 +5001,10 @@
                if (sc->sc_audiopid[n].pid == curproc->p_pid)
                        break;
        }
-       if (n == VAUDIOCHANS)
+       if (n == VAUDIOCHANS) {
+               mutex_exit(sc->sc_lock);
                return false;
+       }
 
        audio_mixer_capture(sc);
        mutex_enter(sc->sc_intr_lock);