Paul Bagyenda wrote:

Hi,

There is a minor problem with the parsing of quoted strings in wap/ wsp_headers.c -- some stacks (nokia/symbian 9 at least) use a literal quote, which then gets missed by the parser. The net effect is that the string returned still retains the quote (without the corresponding end quote).

ok, this was actually already "fixed" in rev 1.17, but I changed it again in 1.18, see my ChangeLog entry:

2004-07-09  Stipe Tolj  <[EMAIL PROTECTED]>
    * gw/smsc/smsc_soap.c: changing #ifdef #warning's to commented blocks to
      get rid of the compiler warnings in the nightly build tests. This module
      still _needs_ revision for the new UUIDs that are used.
    * wap/wsp_headers.c: Bruno moved packing/unpacking of PROXY_AUTHxxx away
      by commenting them out. This was in revision 1.10. And there is no reason
      for this in the ChangeLog entry??? BTW, this makes the
      checks/check_headers.sh test fail for the relevant Proxy-Authxxx headers.
      That is why I put things in again, as long as noone complains and tells
      us why they have been commented in rev 1.10. Along with this Aarno did
      break the quoted parameter parsing within revision 1.17. Reverting this
      back to the previous behaviour to make test/test_headers.c happy again.
    * test/test_header.c: fixing trivial compiler warning
    * test/header_test: chaning the expected values to have quoted parameter
      values to make the test program happy again.

ok, so this way my "fault". Obviously we're rather into fixing the test/test_headers.c instead of the WSP parsing?

Paul, can you confirm that this patch works for all know stacks behaviour?

Commited to cvs, since this seems reasonable to me.

2006-11-18  Stipe Tolj  <stolj at kannel.org>
    * wap/wsp_headers.c: re-adding quote char (") recognizing within the
      wsp_field_value() that isolates header values. Thanks again to Paul.
      [Msg-Id: <[EMAIL PROTECTED]>]

A big clapp-of-hands to Paul!

Stipe

-------------------------------------------------------------------
Kölner Landstrasse 419
40589 Düsseldorf, NRW, Germany

tolj.org system architecture      Kannel Software Foundation (KSF)
http://www.tolj.org/              http://www.kannel.org/

mailto:st_{at}_tolj.org           mailto:stolj_{at}_kannel.org
-------------------------------------------------------------------

Reply via email to