diff options
author | Jeremie Koenig <jk@jk.fr.eu.org> | 2010-08-13 19:05:13 +0200 |
---|---|---|
committer | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2010-08-18 01:25:26 +0200 |
commit | 7e0dc553966e9ac1d0506e8cce9ae3f4708f124b (patch) | |
tree | 90b96cc24d8447c5ed25bc44143e4c876d637a4c | |
parent | bb60526834227fddb97e14bd80fccc405932e352 (diff) |
update_pid_entries(): fix awkwardly indented uninitialized memory leak
* procfs_pid_files.c (update_pid_entries): Replace dynamic allocation of
`stat' with automatic allocation. Memset `stat' to 0.
-rw-r--r-- | procfs_pid_files.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/procfs_pid_files.c b/procfs_pid_files.c index 46861531..d44a2a22 100644 --- a/procfs_pid_files.c +++ b/procfs_pid_files.c @@ -46,14 +46,12 @@ update_pid_entries (struct procfs_dir *dir, const char *name, time_t timestamp, const char *symlink_target) { - struct procfs_dir_entry *dir_entry; - struct stat *stat = (struct stat *) malloc (sizeof (struct stat)); - stat->st_mode = S_IFREG | S_IRUSR | S_IRGRP | S_IROTH; + struct stat stat; - dir_entry = update_entries_list (dir, name, stat, - timestamp, symlink_target); + memset (&stat, 0, sizeof stat); + stat->st_mode = S_IFREG | S_IRUSR | S_IRGRP | S_IROTH; - return dir_entry; + return update_entries_list (dir, name, &stat, timestamp, symlink_target); } /* Creates files to store process information for DIR |