drivers/misc: fix rpmsg dev poll assert crash

when client read and poll wait buffer from server side and server side may poll notify more than one times, then rpmsgdev in client side will call "rpmsgdev_poll_setup(priv, 0, false);" twice which will cause crash in vela rpmsgdev_server.c Signed-off-by: rongyichang <[email protected]>
yamt · Nov 12, 2024 · d73815e · d73815e
1 parent 0bad7f8
commit d73815e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/drivers/misc/rpmsgdev_server.c b/drivers/misc/rpmsgdev_server.c
@@ -403,12 +403,13 @@ static int rpmsgdev_poll_handler(FAR struct rpmsg_endpoint *ept,
     }
   else
     {
-      DEBUGASSERT(dev->cfd != 0);
-
-      msg->header.result = file_poll(&dev->file, &dev->fd, false);
-      if (msg->header.result == OK)
+      if (dev->cfd != 0)
         {
-          dev->cfd = 0;
+          msg->header.result = file_poll(&dev->file, &dev->fd, false);
+          if (msg->header.result == OK)
+            {
+              dev->cfd = 0;
+            }
         }
     }