/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to plugins.d/mandos-client.c

* network-hooks.d/bridge: Use "/usr/sbin/brctl" explicitly.
* plugins.d/mandos-client.c (run_network_hooks): Raise priviliges in
                                                 child process.
  (main): Do not use getuid() to check if running setuid root.  Do not
          raise privileges for run_network_hooks.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
87
87
                                   EX_NOHOST, EX_IOERR, EX_PROTOCOL */
88
88
#include <sys/wait.h>           /* waitpid(), WIFEXITED(),
89
89
                                   WEXITSTATUS(), WTERMSIG() */
 
90
#include <grp.h>                /* setgroups() */
90
91
 
91
92
#ifdef __linux__
92
93
#include <sys/klog.h>           /* klogctl() */
1406
1407
      pid_t hook_pid = fork();
1407
1408
      if(hook_pid == 0){
1408
1409
        /* Child */
 
1410
        /* Raise privileges */
 
1411
        errno = 0;
 
1412
        ret = seteuid(0);
 
1413
        if(ret == -1){
 
1414
          perror_plus("seteuid");
 
1415
        }
 
1416
        /* Raise privileges even more */
 
1417
        errno = 0;
 
1418
        ret = setuid(0);
 
1419
        if(ret == -1){
 
1420
          perror_plus("setuid");
 
1421
        }
 
1422
        /* Set group */
 
1423
        errno = 0;
 
1424
        ret = setgid(0);
 
1425
        if(ret == -1){
 
1426
          perror_plus("setgid");
 
1427
        }
 
1428
        /* Reset supplementary groups */
 
1429
        errno = 0;
 
1430
        ret = setgroups(0, NULL);
 
1431
        if(ret == -1){
 
1432
          perror_plus("setgroups");
 
1433
        }
 
1434
        fprintf_plus(stderr, "Child: getuid() = %d\n", getuid());
 
1435
        fprintf_plus(stderr, "Child: geteuid() = %d\n", geteuid());
1409
1436
        dup2(devnull, STDIN_FILENO);
1410
1437
        close(devnull);
1411
1438
        dup2(STDERR_FILENO, STDOUT_FILENO);
1678
1705
    }
1679
1706
  }
1680
1707
    
1681
 
  if(getuid() == 0){
 
1708
  {
1682
1709
    /* Work around Debian bug #633582:
1683
1710
       <http://bugs.debian.org/633582> */
1684
 
    struct stat st;
1685
1711
    
1686
1712
    /* Re-raise priviliges */
1687
1713
    errno = 0;
1688
1714
    ret = seteuid(0);
1689
1715
    if(ret == -1){
1690
1716
      perror_plus("seteuid");
1691
 
    }
1692
 
    
1693
 
    if(strcmp(seckey, PATHDIR "/" SECKEY) == 0){
1694
 
      int seckey_fd = open(seckey, O_RDONLY);
1695
 
      if(seckey_fd == -1){
1696
 
        perror_plus("open");
1697
 
      } else {
1698
 
        ret = (int)TEMP_FAILURE_RETRY(fstat(seckey_fd, &st));
1699
 
        if(ret == -1){
1700
 
          perror_plus("fstat");
1701
 
        } else {
1702
 
          if(S_ISREG(st.st_mode)
1703
 
             and st.st_uid == 0 and st.st_gid == 0){
1704
 
            ret = fchown(seckey_fd, uid, gid);
1705
 
            if(ret == -1){
1706
 
              perror_plus("fchown");
1707
 
            }
1708
 
          }
1709
 
        }
1710
 
        TEMP_FAILURE_RETRY(close(seckey_fd));
1711
 
      }
1712
 
    }
1713
 
    
1714
 
    if(strcmp(pubkey, PATHDIR "/" PUBKEY) == 0){
1715
 
      int pubkey_fd = open(pubkey, O_RDONLY);
1716
 
      if(pubkey_fd == -1){
1717
 
        perror_plus("open");
1718
 
      } else {
1719
 
        ret = (int)TEMP_FAILURE_RETRY(fstat(pubkey_fd, &st));
1720
 
        if(ret == -1){
1721
 
          perror_plus("fstat");
1722
 
        } else {
1723
 
          if(S_ISREG(st.st_mode)
1724
 
             and st.st_uid == 0 and st.st_gid == 0){
1725
 
            ret = fchown(pubkey_fd, uid, gid);
1726
 
            if(ret == -1){
1727
 
              perror_plus("fchown");
1728
 
            }
1729
 
          }
1730
 
        }
1731
 
        TEMP_FAILURE_RETRY(close(pubkey_fd));
1732
 
      }
1733
 
    }
1734
 
    
1735
 
    /* Lower privileges */
1736
 
    errno = 0;
1737
 
    ret = seteuid(uid);
1738
 
    if(ret == -1){
1739
 
      perror_plus("seteuid");
1740
 
    }
1741
 
  }
1742
 
  
1743
 
  /* Run network hooks */
1744
 
  {
1745
 
    if(getuid() == 0){
1746
 
      /* Re-raise priviliges */
1747
 
      errno = 0;
1748
 
      ret = seteuid(0);
1749
 
      if(ret == -1){
1750
 
        perror_plus("seteuid");
1751
 
      }
1752
 
    }
1753
 
    if(not run_network_hooks("start", interface, delay)){
1754
 
      goto end;
1755
 
    }
1756
 
    if(getuid() == 0){
 
1717
    } else {
 
1718
      struct stat st;
 
1719
      
 
1720
      if(strcmp(seckey, PATHDIR "/" SECKEY) == 0){
 
1721
        int seckey_fd = open(seckey, O_RDONLY);
 
1722
        if(seckey_fd == -1){
 
1723
          perror_plus("open");
 
1724
        } else {
 
1725
          ret = (int)TEMP_FAILURE_RETRY(fstat(seckey_fd, &st));
 
1726
          if(ret == -1){
 
1727
            perror_plus("fstat");
 
1728
          } else {
 
1729
            if(S_ISREG(st.st_mode)
 
1730
               and st.st_uid == 0 and st.st_gid == 0){
 
1731
              ret = fchown(seckey_fd, uid, gid);
 
1732
              if(ret == -1){
 
1733
                perror_plus("fchown");
 
1734
              }
 
1735
            }
 
1736
          }
 
1737
          TEMP_FAILURE_RETRY(close(seckey_fd));
 
1738
        }
 
1739
      }
 
1740
    
 
1741
      if(strcmp(pubkey, PATHDIR "/" PUBKEY) == 0){
 
1742
        int pubkey_fd = open(pubkey, O_RDONLY);
 
1743
        if(pubkey_fd == -1){
 
1744
          perror_plus("open");
 
1745
        } else {
 
1746
          ret = (int)TEMP_FAILURE_RETRY(fstat(pubkey_fd, &st));
 
1747
          if(ret == -1){
 
1748
            perror_plus("fstat");
 
1749
          } else {
 
1750
            if(S_ISREG(st.st_mode)
 
1751
               and st.st_uid == 0 and st.st_gid == 0){
 
1752
              ret = fchown(pubkey_fd, uid, gid);
 
1753
              if(ret == -1){
 
1754
                perror_plus("fchown");
 
1755
              }
 
1756
            }
 
1757
          }
 
1758
          TEMP_FAILURE_RETRY(close(pubkey_fd));
 
1759
        }
 
1760
      }
 
1761
    
1757
1762
      /* Lower privileges */
1758
1763
      errno = 0;
1759
1764
      ret = seteuid(uid);
1763
1768
    }
1764
1769
  }
1765
1770
  
 
1771
  /* Run network hooks */
 
1772
  if(not run_network_hooks("start", interface, delay)){
 
1773
    goto end;
 
1774
  }
 
1775
  
1766
1776
  if(not debug){
1767
1777
    avahi_set_log_function(empty_log);
1768
1778
  }
2217
2227
    }
2218
2228
  }
2219
2229
  
 
2230
  /* Run network hooks */
 
2231
  run_network_hooks("stop", interface, delay);
 
2232
  
2220
2233
  /* Re-raise priviliges */
2221
2234
  {
2222
 
    if(getuid() == 0){
2223
 
      errno = 0;
2224
 
      ret = seteuid(0);
2225
 
      if(ret == -1){
2226
 
        perror_plus("seteuid");
2227
 
      }
 
2235
    errno = 0;
 
2236
    ret = seteuid(0);
 
2237
    if(ret == -1){
 
2238
      perror_plus("seteuid");
2228
2239
    }
2229
2240
    
2230
 
    /* Run network hooks */
2231
 
    run_network_hooks("stop", interface, delay);
2232
 
    
2233
2241
    /* Take down the network interface */
2234
2242
    if(take_down_interface and geteuid() == 0){
2235
2243
      ret = ioctl(sd, SIOCGIFFLAGS, &network);
2248
2256
      }
2249
2257
    }
2250
2258
  }
2251
 
  if(getuid() == 0){
2252
 
    /* Lower privileges permanently */
2253
 
    errno = 0;
2254
 
    ret = setuid(uid);
2255
 
    if(ret == -1){
2256
 
      perror_plus("setuid");
2257
 
    }
 
2259
  /* Lower privileges permanently */
 
2260
  errno = 0;
 
2261
  ret = setuid(uid);
 
2262
  if(ret == -1){
 
2263
    perror_plus("setuid");
2258
2264
  }
2259
2265
  
2260
2266
  /* Removes the GPGME temp directory and all files inside */