accelgroup: Fix a buffer overrun
authorMatthias Clasen <mclasen@redhat.com>
Tue, 24 Dec 2019 04:17:29 +0000 (23:17 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Tue, 24 Dec 2019 04:17:29 +0000 (23:17 -0500)
gtk_accelerator_parse_with_keycode can
overrun its buffer for certain inputs.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/2325
gtk/gtkaccelgroup.c

index f09764ec740e4e029adf7db5bd0ce6dad3442b73..dbf4e261af739e4d20e3e1981246f11ca251a9fe 100644 (file)
@@ -1295,9 +1295,9 @@ gtk_accelerator_parse_with_keycode (const gchar     *accelerator,
               last_ch = *accelerator;
               while (last_ch && last_ch != '>')
                 {
-                  last_ch = *accelerator;
                   accelerator += 1;
                   len -= 1;
+                  last_ch = *accelerator;
                 }
             }
         }