Allow reload command to exist anywhere in the command string

This fixes a crash if you have commands where reload appears in the
middle or at the end, such as `bindsym r mode default, reload`.
This commit is contained in:
Ryan Dwyer 2018-09-01 11:45:48 +10:00
parent ebe65a4d48
commit 7e81e58e7d
3 changed files with 17 additions and 1 deletions

View file

@ -401,3 +401,17 @@ char *argsep(char **stringp, const char *delim) {
found: found:
return start; return start;
} }
const char *strcasestr(const char *haystack, const char *needle) {
size_t needle_len = strlen(needle);
const char *pos = haystack;
const char *end = pos + strlen(haystack) - needle_len;
while (pos <= end) {
if (strncasecmp(pos, needle, needle_len) == 0) {
return pos;
}
++pos;
}
return NULL;
}

View file

@ -46,4 +46,6 @@ char *cmdsep(char **stringp, const char *delim);
// Split string into 2 by delim, handle quotes // Split string into 2 by delim, handle quotes
char *argsep(char **stringp, const char *delim); char *argsep(char **stringp, const char *delim);
const char *strcasestr(const char *haystack, const char *needle);
#endif #endif

View file

@ -310,7 +310,7 @@ void seat_execute_command(struct sway_seat *seat, struct sway_binding *binding)
bool reload = false; bool reload = false;
// if this is a reload command we need to make a duplicate of the // if this is a reload command we need to make a duplicate of the
// binding since it will be gone after the reload has completed. // binding since it will be gone after the reload has completed.
if (strncasecmp(binding->command, "reload", 6) == 0) { if (strcasestr(binding->command, "reload")) {
reload = true; reload = true;
binding_copy = sway_binding_dup(binding); binding_copy = sway_binding_dup(binding);
if (!binding_copy) { if (!binding_copy) {