diff options
author | Michal Bloch <m.bloch@samsung.com> | 2020-12-07 22:16:07 +0100 |
---|---|---|
committer | Karol Lewandowski <k.lewandowsk@samsung.com> | 2020-12-10 09:44:41 +0000 |
commit | 2bcd0ee3fd0273c394c4173acb096ddb88e6f858 (patch) | |
tree | 4791f43652540778865e97867985728cdaab0b78 | |
parent | ed7d431a5572414a2ac3bba640992875f0f8eeec (diff) | |
download | crash-worker-2bcd0ee3fd0273c394c4173acb096ddb88e6f858.tar.gz crash-worker-2bcd0ee3fd0273c394c4173acb096ddb88e6f858.tar.bz2 crash-worker-2bcd0ee3fd0273c394c4173acb096ddb88e6f858.zip |
Fix null dereferences
Change-Id: Id23b9aaad60752f797783f9ccc6f9bbfa9357941
Signed-off-by: Michal Bloch <m.bloch@samsung.com>
(cherry picked from commit a9a8f5e3955f55e03a05878a381257085dc02951)
-rw-r--r-- | src/crash-stack/proc.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/crash-stack/proc.c b/src/crash-stack/proc.c index 46f40e1..8946c21 100644 --- a/src/crash-stack/proc.c +++ b/src/crash-stack/proc.c @@ -294,6 +294,8 @@ char *get_thread_states(const int *tids, int n) { int i; char *res = calloc(1, n); + if (!res) + return NULL; for (i = 0; i < n; ++i) { int state = proc_state(tids[i]); @@ -383,6 +385,9 @@ static int copy_memory_process_vm_readv(int pid, remote_iov = malloc(sizeof(struct iovec)*n_frames); frame_bytes = malloc(sizeof(ssize_t)*n_frames); + if (!local_iov || !remote_iov || !frame_bytes) + goto process_vm_readv_end; + for (i = 0; i < n_frames; ++i) { local_iov[i].iov_base = frames[i]->data; local_iov[i].iov_len = frames[i]->length; |