Changeset b0ee720


Ignore:
Timestamp:
2012-12-02T16:32:59Z (12 years ago)
Author:
Wilmer van der Gaast <wilmer@…>
Branches:
master
Children:
3bda2c2
Parents:
b3eee9b
Message:

Error handling fix in xmltree. Fixes a problem where if the start of the
stream is spread over two packets (or SSL records in this case), xt_handle()
would report a fatal error (which normally means the connection has failed
and should be abandoned immediately).

Returning 1 is likely more correct; just the fact that there's no data to
process yet is not a fatal failure.

This fixes potential issues with Google Talk when using OpenSSL (which
seems to be feeding data on record boundaries, even if additional records
have already been received).

File:
1 edited

Legend:

Unmodified
Added
Removed
  • lib/xmltree.c

    rb3eee9b rb0ee720  
    163163        int i;
    164164       
    165         /* Just in case someone likes infinite loops... */
    166165        if( xt->root == NULL )
    167                 return 0;
     166                return 1;
    168167       
    169168        if( node == NULL )
Note: See TracChangeset for help on using the changeset viewer.