Don't dereference a NULL pointer when we don't have a ':' in the

optarg of the -W option.  Instead, do the next best thing and give a
usage message.

Reported by: kris

1 files changed, 2 insertions, 0 deletions

diff --git a/usr.sbin/raycontrol/raycontrol.c b/usr.sbin/raycontrol/raycontrol.c
index c287643..5f9d5c9 100644
--- a/usr.sbin/raycontrol/raycontrol.c
+++ b/usr.sbin/raycontrol/raycontrol.c
@@ -460,6 +460,8 @@ main(int argc, char *argv[])
 				stringp = optarg;
 				ap = av;
 				*ap = strsep(&stringp, ":");
+				if (stringp == NULL)
+					usage(p);
 				ap++;
 				*ap = strsep(&stringp, ":");
 				mib = atoi(av[0]);