From 4dec659ae5065afe8d449f8f762c704f70c78b17 Mon Sep 17 00:00:00 2001 From: Tim Pepper Date: Tue, 18 Sep 2012 08:17:14 -0700 Subject: [PATCH] Remove extra unlink This was the unlink of the actually processed file. The primary duplicate detection is in move_core(). Due to the two phases of processing in scan_corefolder() it is a fully normal path for a core to get added to the work queue and then a second add attempt likely happens at reprocess_corefile(). Signed-off-by: Tim Pepper --- src/coredump.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/coredump.c b/src/coredump.c index 190938e..ffe2562 100644 --- a/src/coredump.c +++ b/src/coredump.c @@ -728,11 +728,6 @@ static int add_to_processing(char *fullpath) pthread_mutex_lock(&core_status.processing_mtx); if (g_hash_table_lookup(core_status.processing_oops, c2)) { pthread_mutex_unlock(&core_status.processing_mtx); - /* This should only happen when the same core happened - * multiple times in the same second. Go ahead and - * ignore/remove the newer one here */ - fprintf(stderr, "+ ...ignoring/unlinking %s\n", fullpath); - unlink(fullpath); goto clean_add_to_processing; } -- 2.7.4