Re: [PATCH BlueZ 1/3] avctp: Fix dead assignment in control_response

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Tue, Nov 13, 2012 at 3:21 PM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Szymon Janc <szymon.janc@xxxxxxxxx>
>
> Value stored to req is never read before writing it again.
> ---
>  audio/avctp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/audio/avctp.c b/audio/avctp.c
> index 6ba25e4..29756f6 100644
> --- a/audio/avctp.c
> +++ b/audio/avctp.c
> @@ -577,7 +577,7 @@ static void control_response(struct avctp_channel *control,
>                                         size_t operand_count)
>  {
>         struct avctp_pending_req *p = control->p;
> -       struct avctp_control_req *req = p->data;
> +       struct avctp_control_req *req;
>         GSList *l;
>
>         if (p && p->transaction == avctp->transaction) {
> --
> 1.7.11.7

Please ignore this set, it was in my outgoing folder as I just applied
them but they are not part of HFP patchset.


-- 
Luiz Augusto von Dentz
--
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux