wizard: Fix the resolving of the FIXME comment
authorMarcel Hollerbach <marcel-hollerbach@t-online.de>
Fri, 21 Nov 2014 15:26:54 +0000 (10:26 -0500)
committerMike Blumenkrantz <zmike@osg.samsung.com>
Fri, 21 Nov 2014 15:27:27 +0000 (10:27 -0500)
Summary:
Commit rE5b992bdc4a76 added the free for the list, the list is filed
with static pointers from the .c file, soo free will crash.

This pointer does not need to be freed, just freeing the list.

Test Plan: Just run the wizard

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D1679

src/modules/wizard/page_010.c

index 774a24c..da90595 100644 (file)
@@ -145,7 +145,12 @@ wizard_page_init(E_Wizard_Page *pg __UNUSED__, Eina_Bool *need_xdg_desktops __UN
 EAPI int
 wizard_page_shutdown(E_Wizard_Page *pg __UNUSED__)
 {
-   E_FREE_LIST(blang_list, free);
+   void *_tmp;
+   EINA_LIST_FREE(blang_list, _tmp)
+     {
+        // we dont need to free _tmp cause this is just a pointer to the basic_language_predefined_pairs
+        // array. a free will end up in a crash
+     }
    return 1;
 }