[exim-dev] [Bug 2078] exim breaks if current directory doesn't exist

[admin]

https://bugs.exim.org/show_bug.cgi?id=2078

Jeremy Harris  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #4 from Jeremy Harris  ---
Nobody commented

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

[exim-dev] [Bug 2078] exim breaks if current directory doesn't exist

[admin]

https://bugs.exim.org/show_bug.cgi?id=2078

Git Commit  changed:

   What|Removed |Added

 CC||g...@exim.org

--- Comment #3 from Git Commit  ---
Git commit:
https://git.exim.org/exim.git/commitdiff/cd328be98eb5bdda73316eaf95d9d6a617c7df7d

commit cd328be98eb5bdda73316eaf95d9d6a617c7df7d
Author: Jeremy Harris 
AuthorDate: Sat Oct 28 15:09:05 2017 +0100
Commit: Jeremy Harris 
CommitDate: Sat Oct 28 15:09:05 2017 +0100

Do not exit when cwd has no name.  Bug 2078
---
 src/src/exim.c | 9 +++--
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/src/exim.c b/src/src/exim.c
index c12b954..7dd0845 100644
--- a/src/src/exim.c
+++ b/src/src/exim.c
@@ -3793,12 +3793,9 @@ NOTE: immediatly after opening the configuration file we
change the working
 directory to "/"! Later we change to $spool_directory. We do it there, because
 during readconf_main() some expansion takes place already. */

-/* Store the initial cwd before we change directories */
-if ((initial_cwd = os_getcwd(NULL, 0)) == NULL)
-  {
-  perror("exim: can't get the current working directory");
-  exit(EXIT_FAILURE);
-  }
+/* Store the initial cwd before we change directories.  Can be NULL if the
+dir has already been unlinked. */
+initial_cwd = os_getcwd(NULL, 0);

 /* checking:
 -be[m] expansion test-

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

[exim-dev] [Bug 2078] exim breaks if current directory doesn't exist

[admin]

https://bugs.exim.org/show_bug.cgi?id=2078

Jeremy Harris  changed:

   What|Removed |Added

   Assignee|ni...@exim.org  |jgh146...@wizmail.org
 Status|NEW |ASSIGNED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

[exim-dev] [Bug 2078] exim breaks if current directory doesn't exist

[admin]

https://bugs.exim.org/show_bug.cgi?id=2078

Arkadiusz Miskiewicz  changed:

   What|Removed |Added

 CC||ar...@maven.pl

--- Comment #2 from Arkadiusz Miskiewicz  ---
It broke cvslog.pl on old cvs system here where cvs creates temporary
directory, then perl script is run from that directory and directory is then
deleted. Few seconds later tries to send summary email and finish.

Workaround was chdir('/') in perl script.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

[exim-dev] [Bug 2078] exim breaks if current directory doesn't exist

[admin]

https://bugs.exim.org/show_bug.cgi?id=2078

Graeme Fowler  changed:

   What|Removed |Added

 CC||gra...@graemef.net

--- Comment #1 from Graeme Fowler  ---
Commented on exim-dev mailing list but adding here for completeness:

Change was in 4.87 as part of the environment cleanup work, keeping initial_cwd
(and making it available in $initial_cwd).

This is an unusual corner case, IMO. I don't think I've ever come across
something calling an application when it's explicitly not got a CWD in this
way!

Graeme

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##