Limit the alias size -> 256

src/config.c

@@ -369,10 +369,8 @@ init_conf(void)
      cfg_keys    = cfg_getsec(cfg, "keys");
 
      /* alias */
-     if(cfg_size(cfg_alias, "alias"))
+     if(cfg_size(cfg_alias, "alias") < 256)
      {
-          conf.alias = emalloc(cfg_size(cfg_alias, "alias"), sizeof(Alias));
-
           for(i = 0; i < cfg_size(cfg_alias, "alias"); ++i)
           {
                cfgtmp                = cfg_getnsec(cfg_alias, "alias", i);
@@ -380,6 +378,11 @@ init_conf(void)
                conf.alias[i].content = strdup(cfg_getstr(cfgtmp, "content"));
           }
      }
+     else
+     {
+          fprintf(stderr,"WMFS Configuration: Too many alias (%d) !\n", cfg_size(cfg_alias, "alias"));
+          exit(EXIT_FAILURE);
+     }
 
      /* misc */
      conf.font          = alias_to_str(strdup(cfg_getstr(cfg_misc, "font")));

src/event.c

@@ -72,7 +72,7 @@ buttonpress(XButtonEvent *ev)
                               conf.root.mouse[i].func(conf.root.mouse[i].cmd);
 
      /* Tag */
-     for(i = 1; i < conf.ntag[screen_get_sel()]+ 1; ++i)
+     for(i = 1; i < conf.ntag[selscreen] + 1; ++i)
      {
           ITOA(s, i);
           if(ev->window == infobar[selscreen].tags[i]->win)

src/infobar.c

@@ -148,11 +148,9 @@ infobar_draw_taglist(int sc)
 
           if(tags[sc][i].name)
                draw_text(infobar[sc].tags[i]->dr,
-                         PAD / 2,
-                         font->height,
+                         PAD / 2, font->height,
                          ((i == seltag[sc]) ? conf.colors.tagselfg : conf.colors.text),
-                         0,
-                         tags[sc][i].name);
+                         0, tags[sc][i].name);
           barwin_refresh(infobar[sc].tags[i]);
      }
 

src/structs.h

@@ -212,7 +212,7 @@ typedef struct
           MouseBinding *mouse;
           int nmouse;
      } titlebar;
-     Alias *alias;
+     Alias alias[256];
      Tag tag[MAXTAG];
      Layout layout[NUM_OF_LAYOUT];
      int *ntag;

src/wmfs.c

@@ -111,8 +111,6 @@ quit(void)
      free(conf.titlebar.mouse);
      free(conf.client.mouse);
      free(conf.root.mouse);
-     if(conf.alias)
-          free(conf.alias);
      XSync(dpy, False);
 
      return;