The assertion in dispatch_key() can easily be triggered with keys
that map to multi-byte sequences. This patch prevents para_play from
aborting when such a key is pressed. It now issues a warning message,
but no longer aborts.
+again:
+ if (i9ep->key_sequence_length == 0)
+ return 0;
for (i = i9ep->num_key_bindings - 1; i >= 0; i--) {
if (strcmp(i9ep->key_sequence, i9ep->ici->bound_keyseqs[i]))
continue;
for (i = i9ep->num_key_bindings - 1; i >= 0; i--) {
if (strcmp(i9ep->key_sequence, i9ep->ici->bound_keyseqs[i]))
continue;
ret = i9ep->ici->key_handler(i);
return ret < 0? ret : 0;
}
ret = i9ep->ici->key_handler(i);
return ret < 0? ret : 0;
}
+ PARA_WARNING_LOG("ignoring key %d\n", i9ep->key_sequence[0]);
+ /*
+ * We received an undefined key sequence. Throw away the first byte,
+ * and try to parse the remainder.
+ */
+ memmove(i9ep->key_sequence, i9ep->key_sequence + 1,
+ i9ep->key_sequence_length); /* move also terminating zero byte */
+ i9ep->key_sequence_length--;
+ goto again;