[BACK]Return to mandocdb.c CVS log [TXT][DIR] Up to [cvsweb.bsd.lv] / mandoc

Diff for /mandoc/mandocdb.c between version 1.265 and 1.266

version 1.265, 2020/01/26 11:16:47 version 1.266, 2020/01/26 21:25:41
Line 778  treescan(void)
Line 778  treescan(void)
  * See treescan() for the fts(3) version of this.   * See treescan() for the fts(3) version of this.
  */   */
 static void  static void
 filescan(const char *file)  filescan(const char *infile)
 {  {
         char             buf[PATH_MAX];  
         struct stat      st;          struct stat      st;
         struct mlink    *mlink;          struct mlink    *mlink;
         char            *p, *start;          char            *linkfile, *p, *realdir, *start, *usefile;
           size_t           realdir_len;
   
         assert(use_all);          assert(use_all);
   
         if (strncmp(file, "./", 2) == 0)          if (strncmp(infile, "./", 2) == 0)
                 file += 2;                  infile += 2;
   
         /*          /*
          * We have to do lstat(2) before realpath(3) loses           * We have to do lstat(2) before realpath(3) loses
Line 797  filescan(const char *file)
Line 797  filescan(const char *file)
          * we want to use the orginal file name, while for           * we want to use the orginal file name, while for
          * regular files, we want to use the real path.           * regular files, we want to use the real path.
          */           */
         if (lstat(file, &st) == -1) {          if (lstat(infile, &st) == -1) {
                 exitcode = (int)MANDOCLEVEL_BADARG;                  exitcode = (int)MANDOCLEVEL_BADARG;
                 say(file, "&lstat");                  say(infile, "&lstat");
                 return;                  return;
         } else if (S_ISREG(st.st_mode) == 0 && S_ISLNK(st.st_mode) == 0) {          } else if (S_ISREG(st.st_mode) == 0 && S_ISLNK(st.st_mode) == 0) {
                 exitcode = (int)MANDOCLEVEL_BADARG;                  exitcode = (int)MANDOCLEVEL_BADARG;
                 say(file, "Not a regular file");                  say(infile, "Not a regular file");
                 return;                  return;
         }          }
   
Line 811  filescan(const char *file)
Line 811  filescan(const char *file)
          * We have to resolve the file name to the real path           * We have to resolve the file name to the real path
          * in any case for the base directory check.           * in any case for the base directory check.
          */           */
         if (realpath(file, buf) == NULL) {          if ((usefile = realpath(infile, NULL)) == NULL) {
                 exitcode = (int)MANDOCLEVEL_BADARG;                  exitcode = (int)MANDOCLEVEL_BADARG;
                 say(file, "&realpath");                  say(infile, "&realpath");
                 return;                  return;
         }          }
   
         if (op == OP_TEST)          if (op == OP_TEST)
                 start = buf;                  start = usefile;
         else if (strncmp(buf, basedir, basedir_len) == 0)          else if (strncmp(usefile, basedir, basedir_len) == 0)
                 start = buf + basedir_len;                  start = usefile + basedir_len;
 #ifdef HOMEBREWDIR  #ifdef HOMEBREWDIR
         else if (strncmp(buf, HOMEBREWDIR, strlen(HOMEBREWDIR)) == 0)          else if (strncmp(usefile, HOMEBREWDIR, strlen(HOMEBREWDIR)) == 0)
                 start = buf;                  start = usefile;
 #endif  #endif
         else {          else {
                 exitcode = (int)MANDOCLEVEL_BADARG;                  exitcode = (int)MANDOCLEVEL_BADARG;
                 say("", "%s: outside base directory", buf);                  say("", "%s: outside base directory", infile);
                   free(usefile);
                 return;                  return;
         }          }
   
Line 835  filescan(const char *file)
Line 836  filescan(const char *file)
          * Now we are sure the file is inside our tree.           * Now we are sure the file is inside our tree.
          * If it is a symbolic link, ignore the real path           * If it is a symbolic link, ignore the real path
          * and use the original name.           * and use the original name.
          * This implies passing stuff like "cat1/../man1/foo.1"  
          * on the command line won't work.  So don't do that.  
          * Note the stat(2) can still fail if the link target  
          * doesn't exist.  
          */           */
         if (S_ISLNK(st.st_mode)) {          do {
                 if (stat(buf, &st) == -1) {                  if (S_ISLNK(st.st_mode) == 0)
                           break;
   
                   /*
                    * Some implementations of realpath(3) may succeed
                    * even if the target of the link does not exist,
                    * so check again for extra safety.
                    */
                   if (stat(usefile, &st) == -1) {
                         exitcode = (int)MANDOCLEVEL_BADARG;                          exitcode = (int)MANDOCLEVEL_BADARG;
                         say(file, "&stat");                          say(infile, "&stat");
                           free(usefile);
                         return;                          return;
                 }                  }
                 if (strlcpy(buf, file, sizeof(buf)) >= sizeof(buf)) {                  linkfile = mandoc_strdup(infile);
                         say(file, "Filename too long");                  if (op == OP_TEST) {
                         return;                          free(usefile);
                           start = usefile = linkfile;
                           break;
                 }                  }
                 start = buf;                  if (strncmp(infile, basedir, basedir_len) == 0) {
                 if (op != OP_TEST && strncmp(buf, basedir, basedir_len) == 0)                          free(usefile);
                         start += basedir_len;                          usefile = linkfile;
         }                          start = usefile + basedir_len;
                           break;
                   }
   
                   /*
                    * This symbolic link points into the basedir
                    * from the outside.  Let's see whether any of
                    * the parent directories resolve to the basedir.
                    */
                   p = strchr(linkfile, '\0');
                   do {
                           while (*--p != '/')
                                   continue;
                           *p = '\0';
                           if ((realdir = realpath(linkfile, NULL)) == NULL) {
                                   exitcode = (int)MANDOCLEVEL_BADARG;
                                   say(infile, "&realpath");
                                   free(linkfile);
                                   free(usefile);
                                   return;
                           }
                           realdir_len = strlen(realdir) + 1;
                           free(realdir);
                           *p = '/';
                   } while (realdir_len > basedir_len);
   
                   /*
                    * If one of the directories resolves to the basedir,
                    * use the rest of the original name.
                    * Otherwise, the best we can do
                    * is to use the filename pointed to.
                    */
                   if (realdir_len == basedir_len) {
                           free(usefile);
                           usefile = linkfile;
                           start = p + 1;
                   } else {
                           free(linkfile);
                           start = usefile + basedir_len;
                   }
           } while (/* CONSTCOND */ 0);
   
         mlink = mandoc_calloc(1, sizeof(struct mlink));          mlink = mandoc_calloc(1, sizeof(struct mlink));
         mlink->dform = FORM_NONE;          mlink->dform = FORM_NONE;
         if (strlcpy(mlink->file, start, sizeof(mlink->file)) >=          if (strlcpy(mlink->file, start, sizeof(mlink->file)) >=
             sizeof(mlink->file)) {              sizeof(mlink->file)) {
                 say(start, "Filename too long");                  say(start, "Filename too long");
                 free(mlink);                  free(mlink);
                   free(usefile);
                 return;                  return;
         }          }
   
Line 869  filescan(const char *file)
Line 918  filescan(const char *file)
          * but outside our tree, guess the base directory.           * but outside our tree, guess the base directory.
          */           */
   
         if (op == OP_TEST || (start == buf && *start == '/')) {          if (op == OP_TEST || (start == usefile && *start == '/')) {
                 if (strncmp(buf, "man/", 4) == 0)                  if (strncmp(usefile, "man/", 4) == 0)
                         start = buf + 4;                          start = usefile + 4;
                 else if ((start = strstr(buf, "/man/")) != NULL)                  else if ((start = strstr(usefile, "/man/")) != NULL)
                         start += 5;                          start += 5;
                 else                  else
                         start = buf;                          start = usefile;
         }          }
   
         /*          /*
Line 925  filescan(const char *file)
Line 974  filescan(const char *file)
                 *p = '\0';                  *p = '\0';
         }          }
         mlink_add(mlink, &st);          mlink_add(mlink, &st);
           free(usefile);
 }  }
   
 static void  static void

Legend:
Removed from v.1.265  
changed lines
  Added in v.1.266

CVSweb