Interp: Ignore NoSymbol definitions

A NoSymbol syminterp leads to a Any+AnyOfOrAll(None) definition, which
is never what anyone would actually ever want in a sym interp.  The
failure mode here was:
    interpret XF86SomeKeysymYouDontHaveYet {
        action=Something()
    }
leading to:
    interpret Any+AnyOfOrAll(None) {
        action=Something()
    }
leading to every single non-action key triggering your shiny new action
that was only supposed to come from a new keysym.

At least you could VT switch or zap, I guess.

Later, more invasive, revisions will bring a more useful error message.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

1 files changed, 7 insertions, 0 deletions

diff --git a/compat.c b/compat.c
index 144d417..a058dfb 100644
--- a/compat.c
+++ b/compat.c
@@ -658,6 +658,13 @@ HandleInterpDef(InterpDef * def, XkbDescPtr xkb, unsigned merge,
         ACTION("Symbol interpretation ignored\n");
         return True;
     }
+    if (def->sym == NoSymbol)
+    {
+        ERROR("Couldn't determine keysym\n");
+        ACTION("Symbol interpretation ignored\n");
+        return True;
+    }
+
     if (def->merge != MergeDefault)
         merge = def->merge;