* Nils Carlson ([email protected]) wrote:
>
>
> On Wed, 23 Feb 2011, Yannick Brosseau wrote:
>
>> This use of a non-initialized variable was detected by the valgrind test.
>>
>> Signed-off-by: Yannick Brosseau <[email protected]>
>
> Acked-by: Nils Carlson <[email protected]>
Hi Nils,
Please use "Signed-off-by" in the git commit when you are on the merge
path. Otherwise, "Acked-by" would mean that you agree with the patch,
but are not involved in the commit.
Acked-by in the reply here required me to go dig in the git log to see
if it had been merged.
Thanks,
Mathieu
>> ---
>> libustconsumer/libustconsumer.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/libustconsumer/libustconsumer.c
>> b/libustconsumer/libustconsumer.c
>> index 25c84ce..c51b106 100644
>> --- a/libustconsumer/libustconsumer.c
>> +++ b/libustconsumer/libustconsumer.c
>> @@ -621,7 +621,7 @@ static void process_client_cmd(int sock, struct
>> ustcomm_header *req_header,
>> char *recvbuf, struct ustconsumer_instance
>> *instance)
>> {
>> int result;
>> - struct ustcomm_header _res_header;
>> + struct ustcomm_header _res_header = {0};
>> struct ustcomm_header *res_header = &_res_header;
>> struct ustcomm_buffer_info *buf_inf;
>>
>> --
>> 1.7.2.3
>>
>>
>> _______________________________________________
>> ltt-dev mailing list
>> [email protected]
>> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>>
>
> _______________________________________________
> ltt-dev mailing list
> [email protected]
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>
--
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com
_______________________________________________
ltt-dev mailing list
[email protected]
http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
