diff --git a/src/bin/e_config_dialog.c b/src/bin/e_config_dialog.c
index d870dc377..973ecefce 100644
--- a/src/bin/e_config_dialog.c
+++ b/src/bin/e_config_dialog.c
@@ -3,6 +3,13 @@
  */
 #include "e.h"
 
+/* FIXME:
+ * * Having 2 cfdata creates is useless. roll the basic and advanced into 1 and
+ *   simply have 2 different views on the same dataset
+ * * We have no way of associating this config data WITH another object (like
+ *   a window border) so we need a way to pass that in.
+ */
+
 /* local subsystem functions */
 static void _e_config_dialog_free(E_Config_Dialog *cfd);
 static void _e_config_dialog_go(E_Config_Dialog *cfd, E_Config_Dialog_CFData_Type type);
diff --git a/src/bin/e_utils.c b/src/bin/e_utils.c
index 39396bb3d..fffb71bc4 100644
--- a/src/bin/e_utils.c
+++ b/src/bin/e_utils.c
@@ -305,6 +305,8 @@ e_util_edje_icon_list_set(Evas_Object *obj, char *list)
 		  free(buf);
 		  return 1;
 	       }
+	     free(buf);
+	     return 0;
 	  }
      }
    free(buf);