Commit 50467203 authored by Sergey Ryazanov's avatar Sergey Ryazanov Committed by David S. Miller

net: wwan: core: purge rx queue on port close

Purge the rx queue as soon as a user closes the port, just after the
port stop callback invocation. This is to prevent feeding a user that
will open the port next time with outdated and possibly unrelated
data.

While at it also remove the odd skb_queue_purge() call in the port
device destroy callback. The queue will be purged just before the
callback is ivoncated in the wwan_remove_port() function.
Signed-off-by: default avatarSergey Ryazanov <ryazanov.s.a@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c230035c
...@@ -240,7 +240,6 @@ static void wwan_port_destroy(struct device *dev) ...@@ -240,7 +240,6 @@ static void wwan_port_destroy(struct device *dev)
ida_free(&minors, MINOR(port->dev.devt)); ida_free(&minors, MINOR(port->dev.devt));
mutex_destroy(&port->data_lock); mutex_destroy(&port->data_lock);
skb_queue_purge(&port->rxq);
mutex_destroy(&port->ops_lock); mutex_destroy(&port->ops_lock);
kfree(port); kfree(port);
} }
...@@ -462,8 +461,11 @@ static void wwan_port_op_stop(struct wwan_port *port) ...@@ -462,8 +461,11 @@ static void wwan_port_op_stop(struct wwan_port *port)
{ {
mutex_lock(&port->ops_lock); mutex_lock(&port->ops_lock);
port->start_count--; port->start_count--;
if (port->ops && !port->start_count) if (!port->start_count) {
if (port->ops)
port->ops->stop(port); port->ops->stop(port);
skb_queue_purge(&port->rxq);
}
mutex_unlock(&port->ops_lock); mutex_unlock(&port->ops_lock);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment