Re: Openat without die

[Jim Meyering]

Eric Blake wrote:
> On 01/11/2011 12:08 PM, Paul Eggert wrote:
>> On 01/11/11 11:03, Eric Blake wrote:
>>> But you should still drop the xmalloc dependency from the module
>>> description.
>>
>> Hmm, sorry, I guess I should have mentioned that I looked for that,
>> and couldn't find the dependency.  The only modules that mention
>> lib/openat-proc.c are fdopendir and openat, and neither module
>> depends on xmalloc.  I think this is a dependency bug, and that
>> my proposed patch fixes it by removing the need for the dependency.
>
> Hmm, it's an indirect dependency, but it's definitely being pulled in:
>
> $ ./gnulib-tool --with-tests --test openat
> Module list with included dependencies (indented):
...
>     xgetcwd
>
> I'll try looking deeper to see where the real culprit lies, and if we
> can break the rest of this module chain from dragging in xalloc.

save-cwd uses xgetcwd, which calls xalloc_die.

/* Return the current directory, newly allocated.
   Upon an out-of-memory error, call xalloc_die.
   Upon any other type of error, return NULL.  */

char *
xgetcwd (void)
{
  char *cwd = getcwd (NULL, 0);
  if (! cwd && errno == ENOMEM)
    xalloc_die ();
  return cwd;
}