canethgw: Better handling mismatched UDP messages

Don't stop the gw, but print a message.

diff --git a/net/can/canethgw.c b/net/can/canethgw.c
index 532ce9540a9db78ea896835bf487c215de428df0..17fe0dc2a350ed9e52460265e93147aab0a32b40 100644 (file)
--- a/net/can/canethgw.c
+++ b/net/can/canethgw.c
@@ -118,15 +118,21 @@ static int cegw_udp2can(void *data)
                vec[1].iov_len = sizeof(cf);
                ret = kernel_recvmsg(udp_sock, &mh, vec, 2,
                                     sizeof(type) + sizeof(cf), 0);
-               if (ret != sizeof(type) + sizeof(cf))
+               if (ret == 0)
                        break;
+               if (ret != sizeof(type) + sizeof(cf)) {
+                       pr_notice_ratelimited("canethgw: UDP length mismatch\n");
+                       continue;
+               }
 
                type = be16_to_cpu(type);
                switch (type) {
                case CAN_FRAME:
                        cf.can_id = be32_to_cpu(cf.can_id);
                        cegw_can_send(can_sock, &cf);
-               default:;
+                       break;
+               default:
+                       pr_notice_ratelimited("canethgw: Unknown message type\n");
                }
        }