for(mk = mshortcuts; mk < mshortcuts + LEN(mshortcuts); mk++) {
                if(e->xbutton.button == mk->b
-                               && match(mk->mask, e->xbutton.state)) {
+                               && match(mk->mask, e->xbutton.state)
+                               && mk->s) {
                        ttysend(mk->s, strlen(mk->s));
                        return;

mk->s is a string that are assigned in config.h, so I don't think we
should check aginst NULL, because the only way it becomes NULL is
a stupid user who put a NULL in it, and this case let to the luser
get a good BOOOOOM!

What do you think?


-- 
Roberto E. Vargas Caballero
----------------------------
k...@shike2.com
http://www.shike2.com

Reply via email to