Text   |  XML   |  ReML   |   Visible Warnings:

File System Race Condition  at eacces-error.c:65

No properties have been set. | edit properties
Jump to warning location ↓ warning details...
Show Events | Options

maildir_scan_dir

(/home/sate/Testcases/c/cve/dovecot-1.2.0/src/lib-storage/index/maildir/maildir-sync.c)expand/collapse
Show more  
 358  static int maildir_scan_dir(struct maildir_sync_context *ctx, bool new_dir)
 359  {
 360          struct mail_storage *storage = &ctx->mbox->storage->storage;
 361          const char *path;
 362          DIR *dirp;
 363          string_t *src, *dest;
 364          struct dirent *dp;
 365          struct stat st;
 366          enum maildir_uidlist_rec_flag flags;
 367          unsigned int i = 0, move_count = 0;
 368          time_t now;
 369          int ret = 1;
 370          bool move_new, check_touch, dir_changed = FALSE;
 371   
 372          path = new_dir ? ctx->new_dir : ctx->cur_dir;
 373          for (i = 0;; i++) {
 374                  dirp = opendir(path);
 375                  if (dirp != NULL)
 376                          break;
 377   
 378                  if (errno != ENOENT || i == MAILDIR_DELETE_RETRY_COUNT) {
 379                          if (errno == EACCES) {
 380                                  mail_storage_set_critical(storage, "%s",
 381[+]                                         eacces_error_get("opendir", path));
expand/collapse

eacces_error_get

(/home/sate/Testcases/c/cve/dovecot-1.2.0/src/lib/eacces-error.c)expand/collapse
Show more  
 153  const char *eacces_error_get(const char *func, const char *path)
 154  {
 155[+]         return eacces_error_get_full(func, path, FALSE);
expand/collapse

eacces_error_get_full

(/home/sate/Testcases/c/cve/dovecot-1.2.0/src/lib/eacces-error.c)expand/collapse
Show more  
 83  static const char *
 84  eacces_error_get_full(const char *func, const char *path, bool creating)
 85  {
 86          const char *prev_path = path, *dir = "/", *p;
 87          const struct passwd *pw;
 88          const struct group *group;
 89          string_t *errmsg;
 90          struct stat st, dir_st;
 91          int orig_errno, ret = -1;
 92   
 93          orig_errno = errno;
 94          errmsg = t_str_new(256);
 95          str_printfa(errmsg, "%s(%s) failed: Permission denied (euid=%s",
 96                      func, path, dec2str(geteuid()));
 97   
 98          pw = getpwuid(geteuid());
 99          if (pw != NULL)
 100                  str_printfa(errmsg, "(%s)", pw->pw_name);
 101   
 102          str_printfa(errmsg, " egid=%s", dec2str(getegid()));
 103          group = getgrgid(getegid());
 104          if (group != NULL)
 105                  str_printfa(errmsg, "(%s)", group->gr_name);
 106   
 107          while ((p = strrchr(prev_path, '/')) != NULL) {
 108                  dir = t_strdup_until(prev_path, p);
 109                  ret = stat(dir, &st);
 110                  if (ret == 0)
 111                          break;
 112                  if (errno == EACCES) {
 113                          /* see if we have access to parent directory */
 114                  } else if (errno == ENOENT && creating) {
 115                          /* probably mkdir_parents() failed here, find the first 
 116                             parent directory we couldn't create */
 117                  } else {
 118                          /* some other error, can't handle it */
 119                          str_printfa(errmsg, " stat(%s) failed: %m", dir);
 120                          break;
 121                  }
 122                  prev_path = dir;
 123                  dir = "/";
 124                  dir_st = st;
 125          }
 126   
 127          if (ret == 0) {
 128                  /* dir is the first parent directory we can stat() */
 129[+]                 if (test_access(dir, X_OK, errmsg) < 0) {
 130                          if (errno == EACCES)
 131                                  str_printfa(errmsg, " missing +x perm: %s", dir);
 132                  } else if (creating && test_access(dir, W_OK, errmsg) < 0) {
 133                          if (errno == EACCES)
 134                                  str_printfa(errmsg, " missing +w perm: %s", dir);
 135                  } else if (prev_path == path &&
 136[+]                            test_access(path, R_OK, errmsg) < 0) {
expand/collapse

test_access

(/home/sate/Testcases/c/cve/dovecot-1.2.0/src/lib/eacces-error.c)expand/collapse
Show more  
 29  static int test_access(const char *path, int mode, string_t *errmsg)
 30  {
 31          struct stat st;
 32   
 33          if (getuid() == geteuid()) {
 34                  if (access(path, mode) == 0)
 35                          return 0;
 36   
 37                  if (errno != EACCES) {
 38                          str_printfa(errmsg, " access(%s, %d) failed: %m",
 39                                      path, mode);
 40                  }
 41                  return -1;
 42          }  
 43   
 44          /* access() uses real uid, not effective uid.
 45             we'll have to do these checks manually. */
 46          switch (mode) {
 47          case X_OK:
 48                  if (stat(t_strconcat(path, "/test", NULL), &st) == 0)
 49                          return 0;
 50                  if (errno == ENOENT || errno == ENOTDIR)
 51                          return 0;
 52                  if (errno != EACCES)
 53                          str_printfa(errmsg, " stat(%s/test) failed: %m", path);
 54                  return -1;
 55          case R_OK:
 56                  mode = 04;
 57                  break;
 58          case W_OK:
 59                  mode = 02;
 60                  break;
 61          default:
 62                  i_unreached();
 63          }
 64   
 65          if (stat(path, &st) < 0) {
Show more  
Show more  
Show more  
Show more  




Change Warning 7307.25197 : File System Race Condition

Priority:
State:
Finding:
Owner:
Note: