From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: ALSA: firewire-lib: pass the number of data blocks in incoming packets to outgoing packets Date: Tue, 26 May 2015 11:49:24 +0300 Message-ID: <20150526084924.GA25178@mwanda> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from aserp1040.oracle.com (aserp1040.oracle.com [141.146.126.69]) by alsa0.perex.cz (Postfix) with ESMTP id 691CA260690 for ; Tue, 26 May 2015 10:49:42 +0200 (CEST) Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: o-takashi@sakamocchi.jp Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org Hello Takashi Sakamoto, The patch 6fc6b9ce41c6: "ALSA: firewire-lib: pass the number of data blocks in incoming packets to outgoing packets" from May 22, 2015, leads to the following static checker warning: sound/firewire/amdtp.c:857 in_stream_callback() warn: unsigned 'data_blocks' is never less than zero. sound/firewire/amdtp.c 855 856 data_blocks = handle_in_packet(s, payload_quadlets, buffer); 857 if (data_blocks < 0) { ^^^^^^^^^^^^^^^ Never true. But handle_in_packet() does return negative error codes. 858 s->packet_index = -1; 859 break; 860 } 861 regards, dan carpenter