Re: [Qemu-devel] [PATCH] bt: fix use of uninitialized variable seqlen

2015-10-29 Thread Michael Tokarev
19.10.2015 17:08, Paolo Bonzini wrote: > sdp_svc_match, sdp_attr_match and sdp_svc_attr_match read the last > argument. The only sensible way to change the code is to make that last > argument "len" instead of "seqlen" which is the length of a subsequence > in the previous "if" branch. A fun case

[Qemu-devel] [PATCH] bt: fix use of uninitialized variable seqlen

2015-10-19 Thread Paolo Bonzini
sdp_svc_match, sdp_attr_match and sdp_svc_attr_match read the last argument. The only sensible way to change the code is to make that last argument "len" instead of "seqlen" which is the length of a subsequence in the previous "if" branch. To make the structure of the code clearer, use "else" ins