Merge branch 'jk/ident-loosen-getpwuid' into maint
authorJunio C Hamano <gitster@pobox.com>
Mon, 4 Jan 2016 22:02:57 +0000 (14:02 -0800)
committerJunio C Hamano <gitster@pobox.com>
Mon, 4 Jan 2016 22:02:57 +0000 (14:02 -0800)
When getpwuid() on the system returned NULL (e.g. the user is not
in the /etc/passwd file or other uid-to-name mappings), the
codepath to find who the user is to record it in the reflog barfed
and died. Loosen the check in this codepath, which already accepts
questionable ident string (e.g. host part of the e-mail address is
obviously bogus), and in general when we operate fmt_ident() function
in non-strict mode.

* jk/ident-loosen-getpwuid:
ident: loosen getpwuid error in non-strict mode
ident: keep a flag for bogus default_email
ident: make xgetpwuid_self() a static local helper

1  2 
ident.c
diff --combined ident.c
index 00a62e0c4276fb8889a0b158c9e71a22e3be092c,bb1b1743526446bac21abe7da28b15837c8185dc..daf7e1ea8370e5b6a60c5a33a1591ef939655664
+++ b/ident.c
@@@ -10,6 -10,8 +10,8 @@@
  static struct strbuf git_default_name = STRBUF_INIT;
  static struct strbuf git_default_email = STRBUF_INIT;
  static struct strbuf git_default_date = STRBUF_INIT;
+ static int default_email_is_bogus;
+ static int default_name_is_bogus;
  
  #define IDENT_NAME_GIVEN 01
  #define IDENT_MAIL_GIVEN 02
@@@ -23,6 -25,25 +25,25 @@@ static int author_ident_explicitly_give
  #define get_gecos(struct_passwd) ((struct_passwd)->pw_gecos)
  #endif
  
+ static struct passwd *xgetpwuid_self(int *is_bogus)
+ {
+       struct passwd *pw;
+       errno = 0;
+       pw = getpwuid(getuid());
+       if (!pw) {
+               static struct passwd fallback;
+               fallback.pw_name = "unknown";
+ #ifndef NO_GECOS_IN_PWENT
+               fallback.pw_gecos = "Unknown";
+ #endif
+               pw = &fallback;
+               if (is_bogus)
+                       *is_bogus = 1;
+       }
+       return pw;
+ }
  static void copy_gecos(const struct passwd *w, struct strbuf *name)
  {
        char *src;
@@@ -70,48 -91,29 +91,52 @@@ static int add_mailname_host(struct str
        return 0;
  }
  
- static void add_domainname(struct strbuf *out)
 +static int canonical_name(const char *host, struct strbuf *out)
 +{
 +      int status = -1;
 +
 +#ifndef NO_IPV6
 +      struct addrinfo hints, *ai;
 +      memset (&hints, '\0', sizeof (hints));
 +      hints.ai_flags = AI_CANONNAME;
 +      if (!getaddrinfo(host, NULL, &hints, &ai)) {
 +              if (ai && strchr(ai->ai_canonname, '.')) {
 +                      strbuf_addstr(out, ai->ai_canonname);
 +                      status = 0;
 +              }
 +              freeaddrinfo(ai);
 +      }
 +#else
 +      struct hostent *he = gethostbyname(host);
 +      if (he && strchr(he->h_name, '.')) {
 +              strbuf_addstr(out, he->h_name);
 +              status = 0;
 +      }
 +#endif /* NO_IPV6 */
 +
 +      return status;
 +}
 +
+ static void add_domainname(struct strbuf *out, int *is_bogus)
  {
        char buf[1024];
 -      struct hostent *he;
  
        if (gethostname(buf, sizeof(buf))) {
                warning("cannot get host name: %s", strerror(errno));
                strbuf_addstr(out, "(none)");
+               *is_bogus = 1;
                return;
        }
        if (strchr(buf, '.'))
                strbuf_addstr(out, buf);
-       else if (canonical_name(buf, out) < 0)
 -      else if ((he = gethostbyname(buf)) && strchr(he->h_name, '.'))
 -              strbuf_addstr(out, he->h_name);
 -      else {
++      else if (canonical_name(buf, out) < 0) {
                strbuf_addf(out, "%s.(none)", buf);
+               *is_bogus = 1;
+       }
  }
  
- static void copy_email(const struct passwd *pw, struct strbuf *email)
+ static void copy_email(const struct passwd *pw, struct strbuf *email,
+                      int *is_bogus)
  {
        /*
         * Make up a fake email address
  
        if (!add_mailname_host(email))
                return; /* read from "/etc/mailname" (Debian) */
-       add_domainname(email);
+       add_domainname(email, is_bogus);
  }
  
  const char *ident_default_name(void)
  {
        if (!git_default_name.len) {
-               copy_gecos(xgetpwuid_self(), &git_default_name);
+               copy_gecos(xgetpwuid_self(&default_name_is_bogus), &git_default_name);
                strbuf_trim(&git_default_name);
        }
        return git_default_name.buf;
@@@ -144,7 -146,8 +169,8 @@@ const char *ident_default_email(void
                        committer_ident_explicitly_given |= IDENT_MAIL_GIVEN;
                        author_ident_explicitly_given |= IDENT_MAIL_GIVEN;
                } else
-                       copy_email(xgetpwuid_self(), &git_default_email);
+                       copy_email(xgetpwuid_self(&default_email_is_bogus),
+                                  &git_default_email, &default_email_is_bogus);
                strbuf_trim(&git_default_email);
        }
        return git_default_email.buf;
@@@ -332,12 -335,17 +358,17 @@@ const char *fmt_ident(const char *name
                                fputs(env_hint, stderr);
                        die("empty ident name (for <%s>) not allowed", email);
                }
-               pw = xgetpwuid_self();
+               pw = xgetpwuid_self(NULL);
                name = pw->pw_name;
        }
  
-       if (strict && email == git_default_email.buf &&
-           strstr(email, "(none)")) {
+       if (want_name && strict &&
+           name == git_default_name.buf && default_name_is_bogus) {
+               fputs(env_hint, stderr);
+               die("unable to auto-detect name (got '%s')", name);
+       }
+       if (strict && email == git_default_email.buf && default_email_is_bogus) {
                fputs(env_hint, stderr);
                die("unable to auto-detect email address (got '%s')", email);
        }