DO NOT REPLY [Bug 13492] New: - Unintended comma expression in DOMRangeImpl.cpp

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13492>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13492

Unintended comma expression in DOMRangeImpl.cpp

           Summary: Unintended comma expression in DOMRangeImpl.cpp
           Product: Xerces-C++
           Version: Nightly build (please specify the date)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Utilities
        AssignedTo: xerces-c-dev@xml.apache.org
        ReportedBy: dgohman@earthlink.net


The second clause in the for loop in DOMRangeImpl::selectNode is parsed
as a comma expression, not two conditions.

         XMLSize_t i = 0;
-        for (DOMNode* n = parent->getFirstChild(); n!=0, n!=refNode; n =
n->getNextSibling()) {
+        for (DOMNode* n = parent->getFirstChild(); n!=0 && n!=refNode; n =
n->getNextSibling()) {
             i++;
         }

---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-c-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-c-dev-help@xml.apache.org