usbip: keep usbip_device sockfd state in sync with tcp_socket

commit 009f41aed4 upstream.

Keep usbip_device sockfd state in sync with tcp_socket. When tcp_socket
is reset to null, reset sockfd to -1 to keep it in sync.

Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
Cc: stable@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Shuah Khan 2018-01-26 11:56:50 -07:00 коммит произвёл Greg Kroah-Hartman
Родитель d74450a91a
Коммит c276379fa7
2 изменённых файлов: 5 добавлений и 0 удалений

Просмотреть файл

@ -87,6 +87,7 @@ static ssize_t store_sockfd(struct device *dev, struct device_attribute *attr,
goto err; goto err;
sdev->ud.tcp_socket = socket; sdev->ud.tcp_socket = socket;
sdev->ud.sockfd = sockfd;
spin_unlock_irq(&sdev->ud.lock); spin_unlock_irq(&sdev->ud.lock);
@ -186,6 +187,7 @@ static void stub_shutdown_connection(struct usbip_device *ud)
if (ud->tcp_socket) { if (ud->tcp_socket) {
sockfd_put(ud->tcp_socket); sockfd_put(ud->tcp_socket);
ud->tcp_socket = NULL; ud->tcp_socket = NULL;
ud->sockfd = -1;
} }
/* 3. free used data */ /* 3. free used data */
@ -280,6 +282,7 @@ static struct stub_device *stub_device_alloc(struct usb_device *udev)
sdev->ud.status = SDEV_ST_AVAILABLE; sdev->ud.status = SDEV_ST_AVAILABLE;
spin_lock_init(&sdev->ud.lock); spin_lock_init(&sdev->ud.lock);
sdev->ud.tcp_socket = NULL; sdev->ud.tcp_socket = NULL;
sdev->ud.sockfd = -1;
INIT_LIST_HEAD(&sdev->priv_init); INIT_LIST_HEAD(&sdev->priv_init);
INIT_LIST_HEAD(&sdev->priv_tx); INIT_LIST_HEAD(&sdev->priv_tx);

Просмотреть файл

@ -998,6 +998,7 @@ static void vhci_shutdown_connection(struct usbip_device *ud)
if (vdev->ud.tcp_socket) { if (vdev->ud.tcp_socket) {
sockfd_put(vdev->ud.tcp_socket); sockfd_put(vdev->ud.tcp_socket);
vdev->ud.tcp_socket = NULL; vdev->ud.tcp_socket = NULL;
vdev->ud.sockfd = -1;
} }
pr_info("release socket\n"); pr_info("release socket\n");
@ -1044,6 +1045,7 @@ static void vhci_device_reset(struct usbip_device *ud)
if (ud->tcp_socket) { if (ud->tcp_socket) {
sockfd_put(ud->tcp_socket); sockfd_put(ud->tcp_socket);
ud->tcp_socket = NULL; ud->tcp_socket = NULL;
ud->sockfd = -1;
} }
ud->status = VDEV_ST_NULL; ud->status = VDEV_ST_NULL;