These warnings, if they happen, should help users. Signed-off-by: Arturo Borrero Gonzalez <arturo@xxxxxxxxxx> --- src/channel.c | 6 +++++- src/queue_tx.c | 11 +++++++++-- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/src/channel.c b/src/channel.c index acbfa7d..b2f114d 100644 --- a/src/channel.c +++ b/src/channel.c @@ -19,6 +19,7 @@ #include "channel.h" #include "network.h" #include "queue.h" +#include "log.h" static struct channel_ops *ops[CHANNEL_MAX]; extern struct channel_ops channel_mcast; @@ -161,8 +162,11 @@ static void channel_enqueue_errors(struct channel *c) struct channel_error *error; qobj = queue_object_new(Q_ELEM_ERR, sizeof(struct channel_error)); - if (qobj == NULL) + if (qobj == NULL) { + dlog(LOG_WARNING, "could not enqueue channel errors, failed to" + " allocate memory"); return; + } error = (struct channel_error *)qobj->data; error->len = c->buffer->len; diff --git a/src/queue_tx.c b/src/queue_tx.c index 0c99163..83eb111 100644 --- a/src/queue_tx.c +++ b/src/queue_tx.c @@ -22,6 +22,7 @@ #include "queue.h" #include "conntrackd.h" #include "network.h" +#include "log.h" void tx_queue_add_ctlmsg(uint32_t flags, uint32_t from, uint32_t to) { @@ -29,8 +30,11 @@ void tx_queue_add_ctlmsg(uint32_t flags, uint32_t from, uint32_t to) struct nethdr_ack *ack; qobj = queue_object_new(Q_ELEM_CTL, sizeof(struct nethdr_ack)); - if (qobj == NULL) + if (qobj == NULL) { + dlog(LOG_WARNING, "could not queue ACK message. Failed to " + "allocate memory"); return; + } ack = (struct nethdr_ack *)qobj->data; ack->type = NET_T_CTL; @@ -48,8 +52,11 @@ void tx_queue_add_ctlmsg2(uint32_t flags) struct nethdr *ctl; qobj = queue_object_new(Q_ELEM_CTL, sizeof(struct nethdr_ack)); - if (qobj == NULL) + if (qobj == NULL) { + dlog(LOG_WARNING, "could not queue CTL message. Failed to " + "allocate memory"); return; + } ctl = (struct nethdr *)qobj->data; ctl->type = NET_T_CTL; -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html