raster pushed a commit to branch master.

http://git.enlightenment.org/apps/rage.git/commit/?id=40483f4de33d857457db630c5f3ef65fd257f65f

commit 40483f4de33d857457db630c5f3ef65fd257f65f
Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
Date:   Sun Oct 11 14:46:06 2015 +0900

    rage - quieten coverity even though its not an isue
    
    locks in entry frees are not needed as by the time something is freed
    a thread is no longer using it, but this should make coverity happy.
---
 src/bin/browser.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/bin/browser.c b/src/bin/browser.c
index 975e2a2..f53f5f6 100644
--- a/src/bin/browser.c
+++ b/src/bin/browser.c
@@ -500,11 +500,13 @@ _entry_free(Entry *entry)
    Entry *subentry;
    Eina_Stringshare *str;
    if (!entry) return;
+   eina_lock_take(&(entry->lock));
    entry->sels = eina_list_free(entry->sels);
    EINA_LIST_FREE(entry->files, str) eina_stringshare_del(str);
    EINA_LIST_FREE(entry->dirs, subentry) _entry_free(subentry);
    if (entry->base) evas_object_del(entry->base);
    eina_stringshare_del(entry->path);
+   eina_lock_release(&(entry->lock));
    eina_lock_free(&(entry->lock));
    if (entry == selentry) selentry = NULL;
    free(entry);

-- 


Reply via email to