[prev in list] [next in list] [prev in thread] [next in thread] 

List:       axis-c-user
Subject:    Re: Axis 1.6b C++ --- Bug in QName.cpp
From:       Nadir Amra <amra () us ! ibm ! com>
Date:       2006-12-05 4:52:35
Message-ID: OF038AB434.1289771A-ON8625723B.001AB930-8625723B.001ACB45 () us ! ibm ! com
[Download RAW message or body]

Your looking at old code.  The repository location is at 
http://svn.apache.org/viewvc/webservices/axis/.  There is a return 
statement.

Nadir K. Amra


"Chinmoy Bhattacharjee" <chinmoy.bhattacharjee@gmail.com> wrote on 
12/04/2006 08:05:26 PM:

> Hi,
> 
> This looks like a bug in src/xml/QName.cpp
> 
> Notice the code below :-
> 
> void
> QName::splitQNameString(const XML_Ch* qname, XML_Ch sep)
> {
>     XML_Ch *p = const_cast<XML_Ch*>(qname);
>     while (*p)
>     {
>         if (*p == sep)
>         {
>             *p = '\0'; /* put null to separate local name from namespace 
*/ 
>             localname = ++p; /* now p points to localpart */
>             uri = qname; /* qname points to uri */
>             //  -------------- There should be a return here ??? 
> ---------------------------
>         }
>         p++;
>     }
>     /* if there is no separator that means there is no uri */
>     localname = qname; 
>     uri = 0; /* no uri */
> }
> 
> Even if a separator exists, there is no return call from the loop. 
> Hence, the end result will always set the localname to the URI 
> portion. The URL will always be NULL.
> 
> I am not sure what the mergeQNameString is expected to do in the 
> face of the current situation.
> 
> Since this methods are used in determining the size of array 
> structures - array data types do not work.
> 
> - Chinmoy

---------------------------------------------------------------------
To unsubscribe, e-mail: axis-c-user-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-c-user-help@ws.apache.org

[prev in list] [next in list] [prev in thread] [next in thread]