Commit 1be5e58a authored by Greg Kroah-Hartman's avatar Greg Kroah-Hartman Committed by Zheng Yejian
Browse files

USB: move snd_usb_pipe_sanity_check into the USB core 

mainline inclusion
from mainline-v5.10-rc1
commit fcc2cc1f
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IAEPSH

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=fcc2cc1f35613c016e1de25bb001bfdd9eaa25f9



--------------------------------

snd_usb_pipe_sanity_check() is a great function, so let's move it into
the USB core so that other parts of the kernel, including the USB core,
can call it.

Name it usb_pipe_type_check() to match the existing
usb_urb_ep_type_check() call, which now uses this function.

Cc: Jaroslav Kysela <perex@perex.cz>
Cc: "Gustavo A. R. Silva" <gustavoars@kernel.org>
Cc: Eli Billauer <eli.billauer@gmail.com>
Cc: Emiliano Ingrassia <ingrassia@epigenesys.com>
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: Alexander Tsoy <alexander@tsoy.me>
Cc: "Geoffrey D. Bennett" <g@b4.vu>
Cc: Jussi Laako <jussi@sonarnerd.net>
Cc: Nick Kossifidis <mickflemm@gmail.com>
Cc: Dmitry Panchenko <dmitry@d-systems.ee>
Cc: Chris Wulff <crwulff@gmail.com>
Cc: Jesus Ramos <jesus-ramos@live.com>
Reviewed-by: default avatarTakashi Iwai <tiwai@suse.de>
Link: https://lore.kernel.org/r/20200914153756.3412156-2-gregkh@linuxfoundation.org


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Conflicts:
	sound/usb/helper.c
	sound/usb/helper.h
	sound/usb/quirks.c
[Resolve conflicts due to usb-audio sanity check patch
801ebf10 not merged, it
is not relevant to this patch.]
Signed-off-by: default avatarZheng Yejian <zhengyejian1@huawei.com>
parent 8d2f09e2

drivers/usb/core/urb.c

+23 −8
Original line number Diff line number Diff line
@@ -193,24 +193,39 @@ static const int pipetypes[4] = { 
};



/**

 * usb_urb_ep_type_check - sanity check of endpoint in the given urb

 * @urb: urb to be checked

 * usb_pipe_type_check - sanity check of a specific pipe for a usb device

 * @dev: struct usb_device to be checked

 * @pipe: pipe to check

 *

 * This performs a light-weight sanity check for the endpoint in the

 * given urb.  It returns 0 if the urb contains a valid endpoint, otherwise

 * a negative error code.

 * given usb device.  It returns 0 if the pipe is valid for the specific usb

 * device, otherwise a negative error code.

 */

int usb_urb_ep_type_check(const struct urb *urb)

int usb_pipe_type_check(struct usb_device *dev, unsigned int pipe)

{

	const struct usb_host_endpoint *ep;



	ep = usb_pipe_endpoint(urb->dev, urb->pipe);

	ep = usb_pipe_endpoint(dev, pipe);

	if (!ep)

		return -EINVAL;

	if (usb_pipetype(urb->pipe) != pipetypes[usb_endpoint_type(&ep->desc)])

	if (usb_pipetype(pipe) != pipetypes[usb_endpoint_type(&ep->desc)])

		return -EINVAL;

	return 0;

}

EXPORT_SYMBOL_GPL(usb_pipe_type_check);



/**

 * usb_urb_ep_type_check - sanity check of endpoint in the given urb

 * @urb: urb to be checked

 *

 * This performs a light-weight sanity check for the endpoint in the

 * given urb.  It returns 0 if the urb contains a valid endpoint, otherwise

 * a negative error code.

 */

int usb_urb_ep_type_check(const struct urb *urb)

{

	return usb_pipe_type_check(urb->dev, urb->pipe);

}

EXPORT_SYMBOL_GPL(usb_urb_ep_type_check);



/**
@@ -475,7 +490,7 @@ int usb_submit_urb(struct urb *urb, gfp_t mem_flags) 
	 */



	/* Check that the pipe's type matches the endpoint's type */

	if (usb_urb_ep_type_check(urb))

	if (usb_pipe_type_check(urb->dev, urb->pipe))

		dev_WARN(&dev->dev, "BOGUS urb xfer, pipe %x != type %x\n",

			usb_pipetype(urb->pipe), pipetypes[xfertype]);

include/linux/usb.h

+1 −0
Original line number Diff line number Diff line
@@ -1757,6 +1757,7 @@ static inline int usb_urb_dir_out(struct urb *urb) 
	return (urb->transfer_flags & URB_DIR_MASK) == URB_DIR_OUT;

}



int usb_pipe_type_check(struct usb_device *dev, unsigned int pipe);

int usb_urb_ep_type_check(const struct urb *urb);



void *usb_alloc_coherent(struct usb_device *dev, size_t size,