[pkgsrc/pkgsrc-2018Q1]: pkgsrc/audio/mpg321 Pullup ticket #5772 - requested b...

[bsiegert <bsiegert%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/116e152d294d
branches:  pkgsrc-2018Q1
changeset: 309537:116e152d294d
user:      bsiegert <bsiegert%pkgsrc.org@localhost>
date:      Tue Jun 19 12:05:22 2018 +0000

description:
Pullup ticket #5772 - requested by leot
audio/mpg321: bugfix

Revisions pulled up:
- audio/mpg321/Makefile                                         1.20
- audio/mpg321/distinfo                                         1.9
- audio/mpg321/patches/patch-mpg321.c                           1.2

---
   Module Name: pkgsrc
   Committed By:        leot
   Date:                Sat Jun 16 14:43:21 UTC 2018

   Modified Files:
        pkgsrc/audio/mpg321: Makefile distinfo
        pkgsrc/audio/mpg321/patches: patch-mpg321.c

   Log Message:
   mpg321: Do not unlock unititialized main_lock

   Bump PKGREVISION

   Problem spotted by <maya>, fixes PR pkg/53369 by <sevan>.

diffstat:

 audio/mpg321/Makefile               |   3 ++-
 audio/mpg321/distinfo               |   4 ++--
 audio/mpg321/patches/patch-mpg321.c |  14 ++++++++++++--
 3 files changed, 16 insertions(+), 5 deletions(-)

diffs (53 lines):

diff -r e8b22fa571e0 -r 116e152d294d audio/mpg321/Makefile
--- a/audio/mpg321/Makefile     Sun Jun 17 21:06:34 2018 +0000
+++ b/audio/mpg321/Makefile     Tue Jun 19 12:05:22 2018 +0000
@@ -1,7 +1,8 @@
-# $NetBSD: Makefile,v 1.19 2016/06/02 09:18:19 jperkin Exp $
+# $NetBSD: Makefile,v 1.19.18.1 2018/06/19 12:05:22 bsiegert Exp $
 
 DISTNAME=      mpg321_0.3.2.orig
 PKGNAME=       ${DISTNAME:S/_/-/:S/.orig//}
+PKGREVISION=   1
 CATEGORIES=    audio
 MASTER_SITES=  ${MASTER_SITE_SOURCEFORGE:=mpg321/}
 
diff -r e8b22fa571e0 -r 116e152d294d audio/mpg321/distinfo
--- a/audio/mpg321/distinfo     Sun Jun 17 21:06:34 2018 +0000
+++ b/audio/mpg321/distinfo     Tue Jun 19 12:05:22 2018 +0000
@@ -1,8 +1,8 @@
-$NetBSD: distinfo,v 1.8 2016/06/02 09:18:19 jperkin Exp $
+$NetBSD: distinfo,v 1.8.18.1 2018/06/19 12:05:22 bsiegert Exp $
 
 SHA1 (mpg321_0.3.2.orig.tar.gz) = bf1c22542c86af69267828e45f217fdeb49e5d43
 RMD160 (mpg321_0.3.2.orig.tar.gz) = 04ea36c893bbe6fb0054a7fcd71294927cf70ac9
 SHA512 (mpg321_0.3.2.orig.tar.gz) = f1bead2c11e4cde0f1a87e1b2e3d216ef80c9a5dd8b219841961688d44a5fc63a54b7af07359766fde0b2712ddc5d0a90b20149c3228cb2d70e830e15c8ab234
 Size (mpg321_0.3.2.orig.tar.gz) = 151139 bytes
 SHA1 (patch-ao.c) = cb404acdfb032c5a823c717965d14319db0d4466
-SHA1 (patch-mpg321.c) = 0f82c72976ce1f1e7d16cd972eb1e38a0e305f86
+SHA1 (patch-mpg321.c) = c874219aa316899af42d5984ca06860642260d4b
diff -r e8b22fa571e0 -r 116e152d294d audio/mpg321/patches/patch-mpg321.c
--- a/audio/mpg321/patches/patch-mpg321.c       Sun Jun 17 21:06:34 2018 +0000
+++ b/audio/mpg321/patches/patch-mpg321.c       Tue Jun 19 12:05:22 2018 +0000
@@ -1,6 +1,7 @@
-$NetBSD: patch-mpg321.c,v 1.1 2016/06/02 09:18:19 jperkin Exp $
+$NetBSD: patch-mpg321.c,v 1.1.18.1 2018/06/19 12:05:22 bsiegert Exp $
 
-Ensure structs are zero'd before use.
+- Ensure structs are zero'd before use.
+- Do not unlock uninitialized main_lock
 
 --- mpg321.c.orig      2012-03-25 12:27:49.000000000 +0000
 +++ mpg321.c
@@ -12,3 +13,12 @@
      playbuf.pl = pl = new_playlist();
  
      if (!pl)
+@@ -750,8 +751,6 @@ int main(int argc, char *argv[])
+                   if (tcgetattr(0, &terminal_settings) < 0)
+                           perror("tcgetattr()");
+                   memcpy(&old_terminal_settings, &terminal_settings, sizeof(struct termios));
+-                  /* Early thread start */
+-                  sem_post(&main_lock);
+           }
+     }
+     /* Play the mpeg files or zip it! */