env_set.c: remove redundant check

found by cppcheck

[env_set.c:332] -> [env_set.c:333]: (warning) Either the condition '!msg' is redundant or there is possible null pointer dereference: msg.
pull/299/head
Ilya Shipitsin 6 years ago committed by Selva Nair
parent bfcf3b4679
commit ad1640d56d

@ -330,7 +330,7 @@ process_setenv(connection_t *c, UNUSED time_t timestamp, const char *msg)
msg += strlen("setenv "); /* character following "setenv" */ msg += strlen("setenv "); /* character following "setenv" */
msg += strspn(msg, " \t"); /* skip leading space */ msg += strspn(msg, " \t"); /* skip leading space */
if (!msg || msg[0] == '\0') if (msg[0] == '\0')
{ {
WriteStatusLog(c, L"GUI> ", L"Error: Name empty in echo setenv", false); WriteStatusLog(c, L"GUI> ", L"Error: Name empty in echo setenv", false);
return; return;

Loading…
Cancel
Save