diff --git a/src/log-pcap.c b/src/log-pcap.c index e3460b1a3f..52e055478a 100644 --- a/src/log-pcap.c +++ b/src/log-pcap.c @@ -207,13 +207,9 @@ int PcapLogRotateFile(ThreadVars *t, PcapLogData *pl) { SCLogDebug("Removing pcap file %s", pf->filename); if (remove(pf->filename) != 0) { - SCLogError(SC_ERR_PCAP_FILE_DELETE_FAILED, - "failed to remove log file %s: %s", - pf->filename, strerror( errno )); - TAILQ_REMOVE(&pcap_file_list, pf, next); - - PcapFileNameFree(pf); - return -1; + SCLogWarning(SC_ERR_PCAP_FILE_DELETE_FAILED, + "failed to remove log file %s: %s", + pf->filename, strerror( errno )); } else { SCLogDebug("success! removed log file %s", pf->filename); @@ -233,22 +229,17 @@ int PcapLogRotateFile(ThreadVars *t, PcapLogData *pl) { pf->dirname, pfnext->dirname); if (remove(pf->dirname) != 0) { - SCLogError(SC_ERR_PCAP_FILE_DELETE_FAILED, - "failed to remove sguil log %s: %s", - pf->dirname, strerror( errno )); - TAILQ_REMOVE(&pcap_file_list, pf, next); - - PcapFileNameFree(pf); - return -1; + SCLogWarning(SC_ERR_PCAP_FILE_DELETE_FAILED, + "failed to remove sguil log %s: %s", + pf->dirname, strerror( errno )); } } } - - TAILQ_REMOVE(&pcap_file_list, pf, next); - PcapFileNameFree(pf); - - pl->file_cnt--; } + + TAILQ_REMOVE(&pcap_file_list, pf, next); + PcapFileNameFree(pf); + pl->file_cnt--; } if (PcapLogOpenFileCtx(pl) < 0) {