/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

  • Committer: Teddy Hogeborn
  • Date: 2014-06-14 23:43:07 UTC
  • Revision ID: teddy@recompile.se-20140614234307-i0mh9r2n4orkqtki
plugin-runner: Release memory and close FD's correctly in all cases.

* plugins.d/mandos-client.c (main): Init "direntries" to NULL.  Close
                                    "dir_fd" at fallback label.  Do
                                    not free "direntries" if
                                    scandirat() or scandir() failed.
                                    Do free(direntries) at fallback
                                    label.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
1458
1458
  error_t ret_errno = 0;
1459
1459
  if(seteuid(0) == -1){
1460
1460
    ret_errno = errno;
 
1461
    perror_plus("seteuid");
1461
1462
  }
1462
1463
  errno = old_errno;
1463
1464
  return ret_errno;
1474
1475
  }
1475
1476
  if(setuid(0) == -1){
1476
1477
    ret_errno = errno;
 
1478
    perror_plus("seteuid");
1477
1479
  }
1478
1480
  errno = old_errno;
1479
1481
  return ret_errno;
1486
1488
  error_t ret_errno = 0;
1487
1489
  if(seteuid(uid) == -1){
1488
1490
    ret_errno = errno;
 
1491
    perror_plus("seteuid");
1489
1492
  }
1490
1493
  errno = old_errno;
1491
1494
  return ret_errno;
1498
1501
  error_t ret_errno = 0;
1499
1502
  if(setuid(uid) == -1){
1500
1503
    ret_errno = errno;
 
1504
    perror_plus("setuid");
1501
1505
  }
1502
1506
  errno = old_errno;
1503
1507
  return ret_errno;
1550
1554
    if(hook_pid == 0){
1551
1555
      /* Child */
1552
1556
      /* Raise privileges */
1553
 
      errno = raise_privileges_permanently();
1554
 
      if(errno != 0){
 
1557
      if(raise_privileges_permanently() != 0){
1555
1558
        perror_plus("Failed to raise privileges");
1556
1559
        _exit(EX_NOPERM);
1557
1560
      }
1730
1733
    /* Raise privileges */
1731
1734
    ret_errno = raise_privileges();
1732
1735
    if(ret_errno != 0){
1733
 
      errno = ret_errno;
1734
1736
      perror_plus("Failed to raise privileges");
1735
1737
    }
1736
1738
    
1840
1842
    /* Raise privileges */
1841
1843
    ret_errno = raise_privileges();
1842
1844
    if(ret_errno != 0){
1843
 
      errno = ret_errno;
1844
1845
      perror_plus("Failed to raise privileges");
1845
1846
    }
1846
1847
    
2555
2556
  {
2556
2557
    ret_errno = raise_privileges();
2557
2558
    if(ret_errno != 0){
2558
 
      errno = ret_errno;
2559
2559
      perror_plus("Failed to raise privileges");
2560
2560
    } else {
2561
2561
      
2584
2584
    
2585
2585
    ret_errno = lower_privileges_permanently();
2586
2586
    if(ret_errno != 0){
2587
 
      errno = ret_errno;
2588
2587
      perror_plus("Failed to lower privileges permanently");
2589
2588
    }
2590
2589
  }