summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichal Bloch <m.bloch@samsung.com>2020-12-07 22:16:07 +0100
committerKarol Lewandowski <k.lewandowsk@samsung.com>2020-12-10 09:44:51 +0000
commit3c7af7b61aea67731c469405cbae0cbe7c2c7d54 (patch)
tree0aef38faf01caae4009266cbc513707d241bf1ea
parent4712ff053953d2c3320772af3d53ebce302e0dde (diff)
downloadcrash-worker-tizen_5.0.tar.gz
crash-worker-tizen_5.0.tar.bz2
crash-worker-tizen_5.0.zip
Fix null dereferencestizen_5.0
Change-Id: Id23b9aaad60752f797783f9ccc6f9bbfa9357941 Signed-off-by: Michal Bloch <m.bloch@samsung.com> (cherry picked from commit a9a8f5e3955f55e03a05878a381257085dc02951)
-rw-r--r--src/crash-stack/proc.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/src/crash-stack/proc.c b/src/crash-stack/proc.c
index fd346c6..3c9f44a 100644
--- a/src/crash-stack/proc.c
+++ b/src/crash-stack/proc.c
@@ -293,6 +293,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]);
@@ -382,6 +384,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;