rem: if (make_dir (basename_dir, dirpath, mode, NULL))
add: if (! make_dir (basename_dir, dirpath, mode, NULL))
context: make_path (const char *argpath,	   int mode,	   int parent_mode,	   uid_t owner,	   gid_t group,	   int preserve_existing,	   const char *verbose_fmt_string){  struct stat stats;  int retval = 0;  if (stat (argpath, &stats))    {      char *slash;      int tmp_mode;		/* Initial perms for leading dirs.  */      int re_protect;		/* Should leading dirs be unwritable? */      struct ptr_list      {	char *dirname_end;	struct ptr_list *next;      };      struct ptr_list *p, *leading_dirs = NULL;      int do_chdir;		/* Whether to chdir before each mkdir.  */      struct saved_cwd cwd;      char *basename_dir;      char *dirpath;      /* Temporarily relax umask in case it's overly restrictive.  */      mode_t oldmask = umask (0);      /* Make a copy of ARGPATH that we can scribble NULs on.  */      dirpath = (char *) alloca (strlen (argpath) + 1);      strcpy (dirpath, argpath);      strip_trailing_slashes (dirpath);      /* If leading directories shouldn't be writable or executable,	 or should have set[ug]id or sticky bits set and we are setting	 their owners, we need to fix their permissions after making them.  */      if (((parent_mode & WX_USR) != WX_USR)	  || ((owner != (uid_t) -1 || group != (gid_t) -1)	      && (parent_mode & (S_ISUID | S_ISGID | S_ISVTX)) != 0))	{	  tmp_mode = S_IRWXU;	  re_protect = 1;	}      else	{	  tmp_mode = parent_mode;	  re_protect = 0;	}      /* If we can record the current working directory, we may be able	 to do the chdir optimization.  */      do_chdir = !save_cwd (&cwd);      /* If we've saved the cwd and DIRPATH is an absolute pathname,	 we must chdir to `/' in order to enable the chdir optimization.         So if chdir ("/") fails, turn off the optimization.  */      if (do_chdir && *dirpath == '/' && chdir ("/") < 0)	do_chdir = 0;      slash = dirpath;      /* Skip over leading slashes.  */      while (*slash == '/')	slash++;      while (1)	{	  int newly_created_dir;	  int fail;	  /* slash points to the leftmost unprocessed component of dirpath.  */	  basename_dir = slash;	  slash = strchr (slash, '/');	  if (slash == NULL)	    break;	  /* If we're *not* doing chdir before each mkdir, then we have to refer	     to the target using the full (multi-component) directory name.  */	  if (!do_chdir)	    basename_dir = dirpath;	  *slash = '\0';	  fail = make_dir (basename_dir, dirpath, tmp_mode, &newly_created_dir);	  if (fail)	    {	      CLEANUP;	      return 1;	    }	  if (newly_created_dir)	    {	      if (verbose_fmt_string)		error (0, 0, verbose_fmt_string, quote (dirpath));	      if ((owner != (uid_t) -1 || group != (gid_t) -1)		  && chown (basename_dir, owner, group)#if defined AFS && defined EPERM		  && errno != EPERM#endif		  )		{		  error (0, errno, _("cannot change owner and/or group of %s"),			 quote (dirpath));		  CLEANUP;		  return 1;		}	      if (re_protect)		{		  struct ptr_list *new = (struct ptr_list *)		    alloca (sizeof (struct ptr_list));		  new->dirname_end = slash;		  new->next = leading_dirs;		  leading_dirs = new;		}	    }	  /* If we were able to save the initial working directory,	     then we can use chdir to change into each directory before	     creating an entry in that directory.  This avoids making	     stat and mkdir process O(n^2) file name components.  */	  if (do_chdir && chdir (basename_dir) < 0)	    {	      error (0, errno, _("cannot chdir to directory %s"),		     quote (dirpath));	      CLEANUP;	      return 1;	    }	  *slash++ = '/';	  /* Avoid unnecessary calls to `stat' when given	     pathnames containing multiple adjacent slashes.  */	  while (*slash == '/')	    slash++;	}      if (!do_chdir)	basename_dir = dirpath;      /* Done creating leading directories.  Restore original umask.  */      umask (oldmask);      /* We're done making leading directories.	 Create the final component of the path.  */      if (make_dir (basename_dir, dirpath, mode, NULL))	{	  CLEANUP;	  return 1;	}      if (verbose_fmt_string != NULL)	error (0, 0, verbose_fmt_string, quote (dirpath));      if (owner != (uid_t) -1 || group != (gid_t) -1)	{	  if (chown (basename_dir, owner, group)#ifdef AFS	      && errno != EPERM#endif	      )	    {	      error (0, errno, _("cannot change owner and/or group of %s"),		     quote (dirpath));	      retval = 1;	    }	}      /* The above chown may have turned off some permission bits in MODE.	 Another reason we may have to use chmod here is that mkdir(2) is	 required to honor only the file permission bits.  In particular,	 it need not honor the `special' bits, so if MODE includes any	 special bits, set them here.  */      if ((mode & ~S_IRWXUGO)	  && chmod (basename_dir, mode))	{	  error (0, errno, _("cannot change permissions of %s"),		 quote (dirpath));	  retval = 1;	}      CLEANUP_CWD;      /* If the mode for leading directories didn't include owner "wx"	 privileges, we have to reset their protections to the correct	 value.  */      for (p = leading_dirs; p != NULL; p = p->next)	{	  *(p->dirname_end) = '\0';	  if (chmod (dirpath, parent_mode))	    {	      error (0, errno, "cannot change permissions of %s",		     quote (dirpath));	      retval = 1;	    }	}    }  else    {      /* We get here if the entire path already exists.  */      const char *dirpath = argpath;      if (!S_ISDIR (stats.st_mode))	{	  error (0, 0, _("%s exists but is not a directory"), quote (dirpath));	  return 1;	}      if (!preserve_existing)	{	  /* chown must precede chmod because on some systems,	     chown clears the set[ug]id bits for non-superusers,	     resulting in incorrect permissions.	     On System V, users can give away files with chown and then not	     be able to chmod them.  So don't give files away.  */	  if ((owner != (uid_t) -1 || group != (gid_t) -1)	      && chown (dirpath, owner, group)#ifdef AFS	      && errno != EPERM#endif	      )	    {	      error (0, errno, _("cannot change owner and/or group of %s"),		     quote (dirpath));	      retval = 1;	    }	  if (chmod (dirpath, mode))	    {	      error (0, errno, _("cannot change permissions of %s"),				 quote (dirpath));	      retval = 1;	    }	}    }  return retval;}