Re: audio/mocp: add dns pledge to unbreak streaming

Kirill A. Korinsky wrote (2025-02-02 12:45 CET):
> On Sun, 02 Feb 2025 11:40:16 +0100,
> Stefan Hagen <sh+openbsd-ports@codevoid.de> wrote:
> >
> > Hi,
> >
> > mocp crashes when trying to play a web stream:
> >
> > Feb 2 11:38:05.187035: /usr/obj/ports/moc-2.6alpha3/mocp-fb0a63b564b40fa861204e2f804b696c6ee573da/io_curl.c:328 curl_read_internal(): Starting curl...
> > Feb 2 11:38:05.187360: /usr/obj/ports/moc-2.6alpha3/mocp-fb0a63b564b40fa861204e2f804b696c6ee573da/io_curl.c:351 curl_read_internal(): Doing pselect()...
> > mocp[93617]: pledge "dns", syscall 97
> > Abort trap
> >
> > Thus, adding the "dns" pledge with the patch below.
> >
> > OK?
> >
>
> an attempt to run it with your patch as mocp -D -p http://... fails as:
>
> mocp[72461]: pledge "flock", syscall 92
> Abort trap (core dumped)
>
> shall you add it as well?

Sure.

Index: audio/moc/Makefile
===================================================================
RCS file: /cvs/ports/audio/moc/Makefile,v
diff -u -p -r1.34 Makefile
--- audio/moc/Makefile 28 Nov 2024 08:16:37 -0000 1.34
+++ audio/moc/Makefile 2 Feb 2025 15:19:08 -0000
@@ -3,6 +3,7 @@ COMMENT = console audio player
DIST_TUPLE+= gitlab tomaszg mocp fb0a63b564b40fa861204e2f804b696c6ee573da .
PKGNAME = moc-2.6alpha3
CATEGORIES = audio
+REVISION = 0

# GPLv3+
PERMIT_PACKAGE = Yes
Index: audio/moc/patches/patch-main_c
===================================================================
RCS file: /cvs/ports/audio/moc/patches/patch-main_c,v
diff -u -p -r1.5 patch-main_c
--- audio/moc/patches/patch-main_c 25 Nov 2024 11:26:07 -0000 1.5
+++ audio/moc/patches/patch-main_c 2 Feb 2025 15:19:08 -0000
@@ -8,7 +8,7 @@ Index: main.c
if (params->foreground) {
set_me_server ();
server_init (params->debug, params->foreground);
-+ if (pledge("stdio rpath wpath cpath inet unix audio", NULL) == -1)
++ if (pledge("stdio rpath wpath cpath inet dns flock unix audio", NULL) == -1)
+ fatal ("pledge() failed: %s", strerror(errno));
server_loop ();
return;
@@ -17,7 +17,7 @@ Index: main.c
case 0: /* child - start server */
set_me_server ();
server_init (params->debug, params->foreground);
-+ if (pledge("stdio rpath wpath cpath inet unix audio", NULL) == -1)
++ if (pledge("stdio rpath wpath cpath inet dns flock unix audio", NULL) == -1)
+ fatal ("pledge() failed: %s", strerror(errno));
rc = write (notify_pipe[1], &i, sizeof(i));
if (rc < 0)