2 files changed, 86 insertions, 7 deletions

diff --git a/src/firecfg/firecfg.config b/src/firecfg/firecfg.config
index fb996966f..13bcc5110 100644
--- a/src/firecfg/firecfg.config
+++ b/src/firecfg/firecfg.config
@@ -1,7 +1,74 @@
 # /etc/firejail/firecfg.config - firecfg utility configuration file
 # This is the list of programs handled by firecfg utility
 #
+
+# browsers/email
 firefox
 iceweasel
+chromium-browser
+chromium
+conkeror
 thunderbird
+epiphany
+flashpeak-slimjet
+google-chrome-beta
+google-chrome-stable
+google-chrome-unstable
+google-chrome
+icecat
+icedove
+kmail
+midori
+opera-beta
+opera
+qutebrowser
+seamonkey
+seamonkey-bin
+vivaldi-beta
+vivaldi
+
+# bittorrent/ftp
+deluge
+filezilla
+qbittorrent
+rtorrent
+tranmission-gtk
+transmission-qt
+
+# office
+cherrytree
+evince
+fbreader
+localc
+lodraw
+loffice
+lofromtemplate
+loimpress
+lomath
+loweb
+lowriter
+Mathematica
+mathematica
+
+# Media
 vlc
+audacious
+clementine
+deadbeef
+parole
+rhythmbox
+totem
+
+# chat/messaging
+bitlbee
+empathy
+gnome-mplayer
+hexchat
+pidgin
+qtox
+quassel
+xchat
+
+# games
+hedgewars
+wesnot
diff --git a/src/firecfg/main.c b/src/firecfg/main.c
index 7465f2d3e..0c6b278b5 100644
--- a/src/firecfg/main.c
+++ b/src/firecfg/main.c
@@ -30,7 +30,7 @@ static void usage(void) {
         printf("Firecfg is the desktop configuration utility for Firejail software. The utility\n");
         printf("creates several symbolic links to firejail executable. This allows the user to\n");
         printf("sandbox applications automatically, just by clicking on a regular desktop\n");
-        printf("menues and icons.\n\n");
+        printf("menus and icons.\n\n");
         printf("The symbolic links are placed in /usr/local/bin. For more information, see\n");
         printf("DESKTOP INTEGRATION section in man 1 firejail.\n\n");
         printf("Usage: firecfg [OPTIONS]\n\n");
@@ -245,21 +245,33 @@ static void set(void) {
                 lineno++;
                 if (*buf == '#') // comments
                         continue;
+
+                // do not accept .. and/or / in file name
+                if (strstr(buf, "..") || strchr(buf, '/')) {
+                        fprintf(stderr, "Error: invalid line %d in %s\n", lineno, cfgfile);
+                        exit(1);
+                }
                 
                 // remove \n
                 char *ptr = strchr(buf, '\n');
                 if (ptr)
                         *ptr = '\0';
+                        
+                // trim spaces
+                ptr = buf;
+                while (*ptr == ' ' || *ptr == '\t')
+                        ptr++;
+                char *start = ptr;
                 
-                // do not accept .. and/or / in file name
-                if (strstr(buf, "..") || strchr(buf, '/')) {
-                        fprintf(stderr, "Error: invalid line %d in %s\n", lineno, cfgfile);
-                        exit(1);
-                }
+                // empty line
+                if (*start == '\0')
+                        continue;
                 
-                set_file(buf, firejail_exec);
+                // set link
+                set_file(start, firejail_exec);
         }
 
+        fclose(fp);
         free(cfgfile);
         free(firejail_exec);
 }