sndiod: Make opt_setdev() return 1 if the device was accepted

2 files changed, 11 insertions, 8 deletions

diff --git a/usr.bin/sndiod/opt.c b/usr.bin/sndiod/opt.c
index 0dfd8baa33f..17d2456e78d 100644
--- a/usr.bin/sndiod/opt.c
+++ b/usr.bin/sndiod/opt.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: opt.c,v 1.9 2021/11/01 14:43:25 ratchov Exp $	*/
+/*	$OpenBSD: opt.c,v 1.10 2024/04/22 10:42:04 ratchov Exp $	*/
 /*
  * Copyright (c) 2008-2011 Alexandre Ratchov <alex@caoua.org>
  *
@@ -375,7 +375,7 @@ opt_done(struct opt *o)
  * Set opt's device, and (if necessary) move clients to
  * to the new device
  */
-void
+int
 opt_setdev(struct opt *o, struct dev *ndev)
 {
 	struct dev *odev;
@@ -385,12 +385,12 @@ opt_setdev(struct opt *o, struct dev *ndev)
 	int i;
 
 	if (!dev_ref(ndev))
-		return;
+		return 0;
 
 	odev = o->dev;
 	if (odev == ndev) {
 		dev_unref(ndev);
-		return;
+		return 1;
 	}
 
 	/* check if clients can use new device */
@@ -399,18 +399,20 @@ opt_setdev(struct opt *o, struct dev *ndev)
 			continue;
 		if (s->ops != NULL && !dev_iscompat(odev, ndev)) {
 			dev_unref(ndev);
-			return;
+			return 0;
 		}
 	}
 
 	/*
 	 * if we're using MMC, move all opts to the new device, mtc_setdev()
 	 * will call us back
+	 *
+	 * XXX: move this to the end to avoid the recursion
 	 */
 	if (o->mtc != NULL && o->mtc->dev != ndev) {
 		mtc_setdev(o->mtc, ndev);
 		dev_unref(ndev);
-		return;
+		return 1;
 	}
 
 	c = ctl_find(CTL_OPT_DEV, o, o->dev);
@@ -468,6 +470,7 @@ opt_setdev(struct opt *o, struct dev *ndev)
 	}
 
 	dev_unref(ndev);
+	return 1;
 }
 
 /*
diff --git a/usr.bin/sndiod/opt.h b/usr.bin/sndiod/opt.h
index 7dd9e2f7c71..386e6c2c565 100644
--- a/usr.bin/sndiod/opt.h
+++ b/usr.bin/sndiod/opt.h
@@ -1,4 +1,4 @@
-/*	$OpenBSD: opt.h,v 1.7 2021/11/01 14:43:25 ratchov Exp $	*/
+/*	$OpenBSD: opt.h,v 1.8 2024/04/22 10:42:04 ratchov Exp $	*/
 /*
  * Copyright (c) 2008-2012 Alexandre Ratchov <alex@caoua.org>
  *
@@ -47,7 +47,7 @@ struct opt *opt_byname(char *);
 struct opt *opt_bynum(int);
 void opt_init(struct opt *);
 void opt_done(struct opt *);
-void opt_setdev(struct opt *, struct dev *);
+int opt_setdev(struct opt *, struct dev *);
 struct dev *opt_ref(struct opt *);
 void opt_unref(struct opt *);