[PATCH 2/2] Connector: Correctly set the error code in case of success when dispatching receive callbacks

From: K. Y. Srinivasan
Date: Tue May 17 2011 - 18:11:47 EST

The recent changes to the connector code introduced this bug where even
when a callback was invoked, we would return an error resulting in
double freeing of the skb. This patch fixes this bug.

Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
drivers/connector/connector.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/connector/connector.c b/drivers/connector/connector.c
index d770058..c36fbe1 100644
--- a/drivers/connector/connector.c
+++ b/drivers/connector/connector.c
@@ -139,6 +139,7 @@ static int cn_call_callback(struct sk_buff *skb)

if (cbq != NULL) {
+ err = 0;
cbq->callback(msg, nsp);

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/