uxc: fix potential NULL-pointer dereference

Check for NULL returned by strrchr() and return error in that case.
It should not be reachable as the string should always contain a '/'
character, but follow best practices anyway.

Coverity CID: 1500356 Dereference null return value
Signed-off-by: Daniel Golle <daniel@makrotopia.org>

1 files changed, 3 insertions, 0 deletions

diff --git a/uxc.c b/uxc.c
index 332b91d..c15b4bc 100644
--- a/uxc.c
+++ b/uxc.c
@@ -1032,6 +1032,9 @@ static int uxc_set(char *name, char *path, signed char autostart, char *pidfile,
 			char *t1, *t2;
 			t1 = strdup(cfname);
 			t2 = strrchr(t1, '/');
+			if (!t2)
+				return -EINVAL;
+
 			*t2 = '\0';
 
 			if (asprintf(&t2, "%s/settings", t1) == -1)