[sheepdog] [PATCH] bs_sheepdog.c: add missing initialization of read data length

Hitoshi Mitake mitake.hitoshi at gmail.com
Thu Dec 19 08:01:09 CET 2013


At Thu, 19 Dec 2013 14:46:58 +0900,
Ryusuke Konishi wrote:
> 
> In the sheepdog driver, the read data length argument of do_req
> function is not initialized by some caller functions such as
> sd_sync(), sd_close(), and create_branch().
> 
> This patch adds those missing initializations to prevent uncertain
> behavior in do_req() and fixes the following warning of valgrind:
> 
>  ==1885== Conditional jump or move depends on uninitialised value(s)
>  ==1885==    at 0x425994: do_req (bs_sheepdog.c:598)
>  ==1885==    by 0x426A06: bs_sheepdog_close (bs_sheepdog.c:1100)
>  ==1885==    by 0x4142B4: tgt_device_destroy (target.c:705)
>  ==1885==    by 0x412335: tgt_mgmt (mgmt.c:222)
>  ==1885==    by 0x412776: mtask_handler (mgmt.c:494)
>  ==1885==    by 0x4106A8: event_loop (tgtd.c:411)
>  ==1885==    by 0x410D64: main (tgtd.c:583)
> 
> Signed-off-by: Ryusuke Konishi <konishi.ryusuke at lab.ntt.co.jp>
> Cc: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
> ---
>  usr/bs_sheepdog.c |    6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Looks good to me.
Reviewed-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>

Thanks,
Hitoshi



More information about the sheepdog mailing list