/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-07-12 13:13:28 UTC
  • Revision ID: teddy@recompile.se-20140712131328-gdkvcea7f9rov7bn
mandos-client: Fix minor memory leak on memory full or clock failure.

* plugins.d/mandos-client.c (add_server): On any errors, free
                                          new_server and its contents
                                          before returning.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugins.d/mandos-client.c
234
234
                          .af = af };
235
235
  if(new_server->ip == NULL){
236
236
    perror_plus("strdup");
 
237
    free(new_server);
237
238
    return false;
238
239
  }
239
240
  ret = clock_gettime(CLOCK_MONOTONIC, &(new_server->last_seen));
240
241
  if(ret == -1){
241
242
    perror_plus("clock_gettime");
 
243
    free(new_server->ip);
 
244
    free(new_server);
242
245
    return false;
243
246
  }
244
247
  /* Special case of first server */
1458
1461
  error_t ret_errno = 0;
1459
1462
  if(seteuid(0) == -1){
1460
1463
    ret_errno = errno;
1461
 
    perror_plus("seteuid");
1462
1464
  }
1463
1465
  errno = old_errno;
1464
1466
  return ret_errno;
1475
1477
  }
1476
1478
  if(setuid(0) == -1){
1477
1479
    ret_errno = errno;
1478
 
    perror_plus("seteuid");
1479
1480
  }
1480
1481
  errno = old_errno;
1481
1482
  return ret_errno;
1488
1489
  error_t ret_errno = 0;
1489
1490
  if(seteuid(uid) == -1){
1490
1491
    ret_errno = errno;
1491
 
    perror_plus("seteuid");
1492
1492
  }
1493
1493
  errno = old_errno;
1494
1494
  return ret_errno;
1501
1501
  error_t ret_errno = 0;
1502
1502
  if(setuid(uid) == -1){
1503
1503
    ret_errno = errno;
1504
 
    perror_plus("setuid");
1505
1504
  }
1506
1505
  errno = old_errno;
1507
1506
  return ret_errno;
1510
1509
__attribute__((nonnull))
1511
1510
void run_network_hooks(const char *mode, const char *interface,
1512
1511
                       const float delay){
1513
 
  struct dirent **direntries;
 
1512
  struct dirent **direntries = NULL;
1514
1513
  if(hookdir_fd == -1){
1515
1514
    hookdir_fd = open(hookdir, O_RDONLY);
1516
1515
    if(hookdir_fd == -1){
1554
1553
    if(hook_pid == 0){
1555
1554
      /* Child */
1556
1555
      /* Raise privileges */
1557
 
      if(raise_privileges_permanently() != 0){
 
1556
      errno = raise_privileges_permanently();
 
1557
      if(errno != 0){
1558
1558
        perror_plus("Failed to raise privileges");
1559
1559
        _exit(EX_NOPERM);
1560
1560
      }
1670
1670
                   direntry->d_name);
1671
1671
    }
1672
1672
  }
 
1673
  free(direntries);
1673
1674
  if((int)TEMP_FAILURE_RETRY(close(hookdir_fd)) == -1){
1674
1675
    perror_plus("close");
1675
1676
  } else {
1732
1733
    /* Raise privileges */
1733
1734
    ret_errno = raise_privileges();
1734
1735
    if(ret_errno != 0){
 
1736
      errno = ret_errno;
1735
1737
      perror_plus("Failed to raise privileges");
1736
1738
    }
1737
1739
    
1841
1843
    /* Raise privileges */
1842
1844
    ret_errno = raise_privileges();
1843
1845
    if(ret_errno != 0){
 
1846
      errno = ret_errno;
1844
1847
      perror_plus("Failed to raise privileges");
1845
1848
    }
1846
1849
    
2274
2277
      }
2275
2278
      free(direntries);
2276
2279
    } else {
2277
 
      free(direntries);
 
2280
      if(ret == 0){
 
2281
        free(direntries);
 
2282
      }
2278
2283
      fprintf_plus(stderr, "Could not find a network interface\n");
2279
2284
      exitcode = EXIT_FAILURE;
2280
2285
      goto end;
2553
2558
  {
2554
2559
    ret_errno = raise_privileges();
2555
2560
    if(ret_errno != 0){
 
2561
      errno = ret_errno;
2556
2562
      perror_plus("Failed to raise privileges");
2557
2563
    } else {
2558
2564
      
2581
2587
    
2582
2588
    ret_errno = lower_privileges_permanently();
2583
2589
    if(ret_errno != 0){
 
2590
      errno = ret_errno;
2584
2591
      perror_plus("Failed to lower privileges permanently");
2585
2592
    }
2586
2593
  }
2608
2615
      int numentries = scandir(tempdir, &direntries, notdotentries,
2609
2616
                               alphasort);
2610
2617
#endif  /* not __GLIBC__ */
2611
 
      if(numentries > 0){
 
2618
      if(numentries >= 0){
2612
2619
        for(int i = 0; i < numentries; i++){
2613
2620
          ret = unlinkat(tempdir_fd, direntries[i]->d_name, 0);
2614
2621
          if(ret == -1){