AaronZhangL / unimrcp

Automatically exported from code.google.com/p/unimrcp
Apache License 2.0
0 stars 0 forks source link

apt_header_field_create_from_line() stops processing on a semicolon character #88

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi Arsen,

There is a small problem in apt_header_field_create_from_line(), I 
believe. When you call apt_text_field_read() for the second time (to get 
the header value) you ask it to stop on a semicolon:

    /* read value */
    apt_text_field_read(&stream,';',TRUE,&item);

This will truncate the value of a VENDOR-SPECIFIC-PARAMETERS which 
contains semicolon separated strings, to the value of the first string.

To circumvent this problem, I passed a non-printable character as an 
argument to apt_text_field_read(), e.g.:

    apt_text_field_read(&stream,0,TRUE,&item);

What do you think?

Thanks and best regards.

This is probably r1654, or close.

Original issue reported on code.google.com by Curat...@gmail.com on 6 May 2010 at 12:11

GoogleCodeExporter commented 9 years ago
Hi Vlad,

Thanks for the catch. It was a bit stupid to use ';' there. Your suggestion 
looks
good. Fixed in r1685.

Best regards,
Arsen

Original comment by achalo...@gmail.com on 6 May 2010 at 5:38