tree ffd22203a251cc7d2587fd826cb758212ccfd4bd
parent 09455a76d07f7f57c41aeaaf1447096feebbf244
author Zach Reizner <zachr@google.com> 1521694240 -0700
committer Tim Detwiler <tjdetwiler@google.com> 1539128144 -0700

CHROMIUM: virtwl: properly free qentries that are empty

If a qentry was received from the device empty, and an ioctl recv is
performed on the qentry, the vfd_qentry_free_if_empty call would be
skipped over. Because the qentry is still part of the vfd, the ioctl
recv would assume there is more data to read and repeat the entire
proces, causing a hang.

The solution in this change is to always call vfd_qentry_free_if_empty
even if the entry is empty so that the entry gets removed from the vfd
and the ioctl recv can return or enter a wait queue.

BUG=chromium:791724
TEST=test code from BUG

Change-Id: Ifb744dff6a56ac104b9f0816dee680461824daba
Signed-off-by: Zach Reizner <zachr@google.com>
Reviewed-on: https://chromium-review.googlesource.com/974894
Commit-Ready: Zach Reizner <zachr@chromium.org>
Tested-by: Zach Reizner <zachr@chromium.org>
Reviewed-by: Dmitry Torokhov <dtor@chromium.org>