signals: Don't forget to allocate SiginfoEntry

The se variable is just an array of pointers on these
objects. Need to allocate the objects themselves.

Signed-off-by: Pavel Emelyanov <xemul@parallels.com>

1 files changed, 15 insertions, 2 deletions

diff --git a/cr-dump.c b/cr-dump.c
index a660c14c7..baedc98c2 100644
--- a/cr-dump.c
+++ b/cr-dump.c
@@ -1202,10 +1202,23 @@ static int dump_signal_queue(pid_t tid, SignalQueueEntry **sqe, bool group)
 		}
 
 		for (j = queue->n_signals - nr; j < queue->n_signals; j++) {
-			queue->signals[j]->siginfo.len = sizeof(siginfo_t);
-			queue->signals[j]->siginfo.data = (void *) (siginfo + j);
+			SiginfoEntry *se;
+
+			se = xmalloc(sizeof(*se));
+			if (!se) {
+				ret = -1;
+				break;
+			}
+
+			siginfo_entry__init(se);
+			se->siginfo.len = sizeof(siginfo_t);
+			se->siginfo.data = (void *) (siginfo + j);
+			queue->signals[j] = se;
 		}
 
+		if (ret < 0)
+			break;
+
 		arg.off += nr;
 	}