cmd: date: Do not overwrite arguments
authorRoman Kapl <rka@sysgo.com>
Fri, 8 Feb 2019 09:01:02 +0000 (10:01 +0100)
committerTom Rini <trini@konsulko.com>
Tue, 19 Feb 2019 13:55:43 +0000 (08:55 -0500)
Arguments are const and belong to the caller. Calling date in a hush
loop will yield different results from the second invocation.

Signed-off-by: Roman Kapl <rka@sysgo.com>
cmd/date.c

index 1115b6c..7fa950a 100644 (file)
@@ -159,18 +159,18 @@ int mk_date (const char *datestr, struct rtc_time *tmp)
        int len, val;
        char *ptr;
 
-       ptr = strchr (datestr,'.');
-       len = strlen (datestr);
+       ptr = strchr(datestr, '.');
+       len = strlen(datestr);
 
        /* Set seconds */
        if (ptr) {
                int sec;
 
-               *ptr++ = '\0';
+               ptr++;
                if ((len - (ptr - datestr)) != 2)
                        return (-1);
 
-               len = strlen (datestr);
+               len -= 3;
 
                if (cnvrt2 (ptr, &sec))
                        return (-1);