Re: [PATCH] LMAIL: preventing of excessive hdd access
Stephan Mueller <[email protected]>
| Newsgroups | gmane.mail.xmail.general |
|---|---|
| Message-ID | <1274873714.30052.130.camel@tauon> |
An update of the patch: I forgot to also close the directory file handle which leads to a file descriptor leak. Furthermore, I removed the SysRemove() call as I think it is not necessary any more - the file is never created if it is not needed. and If it is needed, it should not be created. Ciao Stephan Am Mittwoch, den 26.05.2010, 11:42 +0200 schrieb Stephan Mueller: > Hi, > > please see attached patch for LMAIL. It stops from creating tmp files if > there is no need to do so. This prevents excessive hdd access. > > Though, there is one very minor issue - maybe you can help me: Since > SysRemove(pszSSFileName); is always called it usually calls unlink() > which returns in an strace an ENOENT (as the tmp file most of the time > does not exist). I think that this issue is negligible. Still, Davide, > can you please check whether we can simply remove that SysRemove() call > as the tmp file should have never been created with my patch when > iFileCount == 0? > > Thanks > Stephan > > _______________________________________________ > xmail mailing list > [email protected] > http://xmailserver.org/mailman/listinfo/xmail _______________________________________________ xmail mailing list [email protected] http://xmailserver.org/mailman/listinfo/xmail
500-LMAILSrv.patch
(text/x-patch, 1.2 KB)
--- LMAILSvr.cpp.orig 2010-05-26 08:32:53.381926212 +0200
+++ LMAILSvr.cpp 2010-05-26 13:31:52.271926317 +0200
@@ -116,6 +116,7 @@
int iMaxSSFileName)
{
char szSpoolDir[SYS_MAX_PATH];
+ FILE *pSSFile = NULL;
LMAILGetSpoolDir(szSpoolDir, sizeof(szSpoolDir));
@@ -129,14 +130,6 @@
UsrGetTmpFile(NULL, pszSSFileName, iMaxSSFileName);
- FILE *pSSFile = fopen(pszSSFileName, "wb");
-
- if (pSSFile == NULL) {
- ErrorPush();
- RLckUnlockSH(hResLock);
- return ErrorPop();
- }
-
int iFileCount = 0;
char szSpoolFileName[SYS_MAX_PATH];
FSCAN_HANDLE hFileScan = MscFirstFile(szSpoolDir, 0, szSpoolFileName,
@@ -149,6 +142,17 @@
if ((ulHashValue % (unsigned long) pLMAILCfg->lNumThreads) ==
(unsigned long) lThreadId) {
+ if(pSSFile == NULL) {
+ pSSFile = fopen(pszSSFileName, "wb");
+
+ if (pSSFile == NULL) {
+ ErrorPush();
+ MscCloseFindFile(hFileScan);
+ RLckUnlockSH(hResLock);
+ return ErrorPop();
+ }
+ }
+
fprintf(pSSFile, "%s\r\n", szSpoolFileName);
++iFileCount;
}
@@ -159,7 +163,6 @@
RLckUnlockSH(hResLock);
if (iFileCount == 0) {
- SysRemove(pszSSFileName);
SetEmptyString(pszSSFileName);
ErrSetErrorCode(ERR_NO_LOCAL_SPOOL_FILES);