These warnings, if they happen, should help users.
Signed-off-by: Arturo Borrero Gonzalez <[email protected]>
---
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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
