Hi Johan, > The convention of the L2CAP code is to return EFAULT when a CID in the > request packet is invalid. This patch fixes the l2cap_config_req to use > that error code instead of ENOENT. > > Signed-off-by: Johan Hedberg <johan.hedberg@xxxxxxxxx> > --- > net/bluetooth/l2cap_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c > index 9ec1561f..d134501 100644 > --- a/net/bluetooth/l2cap_core.c > +++ b/net/bluetooth/l2cap_core.c > @@ -3978,7 +3978,7 @@ static inline int l2cap_config_req(struct l2cap_conn *conn, > > chan = l2cap_get_chan_by_scid(conn, dcid); > if (!chan) > - return -ENOENT; > + return -EFAULT; do we really want to use EFAULT. That is normally only used if a memory copy from userspace to kernel or vice versa fails. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
