On 2021-09-12 08:34, Eric Kohl wrote:
> diff --git a/drivers/hid/kbdhid/kbdhid.c b/drivers/hid/kbdhid/kbdhid.c
> index bacf72a0109..b07a7d8445c 100644
> --- a/drivers/hid/kbdhid/kbdhid.c
> +++ b/drivers/hid/kbdhid/kbdhid.c
> @@ -444,8 +444,8 @@ KbdHid_InternalDeviceControl(
> case IOCTL_KEYBOARD_QUERY_INDICATORS:
> if (IoStack->Parameters.DeviceIoControl.OutputBufferLength < sizeof(KEYBOARD_INDICATOR_PARAMETERS))
> {
> - /* invalid parameter */
> - Irp->IoStatus.Status = STATUS_INVALID_PARAMETER;
> + /* buffer too small */
> + Irp->IoStatus.Status = STATUS_BUFFER_TOO_SMALL;
> IoCompleteRequest(Irp, IO_NO_INCREMENT);
> return STATUS_INVALID_PARAMETER;
Shouldn't the dispatch routine's return value change accordingly in
these cases?
Thanks,
Thomas