[Chilli] [PATCH] Binstatusfile not moved after chilliredir fork - Rev. 257

Alberto Bellettato albesvs at yahoo.it
Tue Dec 15 18:13:31 UTC 2009


In rev. 257 there is a problem with the binstatusfile, that issues a message in the log files "(Directory not empty) /tmp/chilli-xxxxx".

That is caused by the rename(file,file2) not working (not moving the bin file) because file = file2 = file with old PID.
Infact file2 is set by chilli_binconfig(), that used the "bc = _options.binconfig" value (set to the old PID).

I think the /tmp/chilli-xxxxx dir should be clean when chilli stop too, but it is not a priority since it uses only 16 bytes.

Here is the patch.

Index: src/chilli.c
===================================================================
--- src/chilli.c        (revisione 257)
+++ src/chilli.c        (copia locale)
@@ -4170,9 +4170,9 @@
        log_err(errno, file2);

       chilli_binconfig(file, sizeof(file), cpid);
-      chilli_binconfig(file2, sizeof(file2), getpid());
+      snprintf(file2, sizeof(file2), "/tmp/chilli-%d/config.bin", getpid());

-      rename(file, file2);
+      if (rename(file, file2)) log_err(errno, file);

       umask(process_mask);




More information about the Chilli mailing list