Opened 5 years ago

Closed 5 years ago

#1653 closed defect (fixed)

SAXParserException 'Content not allowed in trailing section'

Reported by: kay Owned by: zzz
Priority: minor Milestone: 0.9.22
Component: router/transport Version: 0.9.21
Keywords: Cc:
Parent Tickets: #481 Sensitive: no

Description

In order to prevent SAXParserException 'Content not allowed in trailing section'
all three read() methods of java.io.FilterInputStream? have to be overridden.

Patch attached.

ant buildclean and ant fulltest passed successfully and
the patched router runs fine without the above mentioned exception.

Please review.

Subtickets

Attachments (1)

0001-In-order-to-prevent-SAXParserException-Content-not-a.patch​ (1.9 KB) - added by kay 5 years ago.
patch for this ticket

Download all attachments as: .zip

Change History (4)

Changed 5 years ago by kay

Attachment: 0001-In-order-to-prevent-SAXParserException-Content-not-a.patch​ added

patch for this ticket

comment:1 Changed 5 years ago by zzz

Component: router/general → router/transport
Milestone: undecided → 0.9.22
Owner: set to zzz
Parent Tickets: → 481
Status: new → accepted

comment:2 Changed 5 years ago by zzz

Looks good, great catch!

If you copied the code in your patch from somewhere, please specify source and license.

If you wrote it yourself, please agree to our dev agreement from ​http://i2p-projekt.i2p/en/get-involved/develop/licenses#commit :

    Unless marked otherwise, all code I commit is implicitly licensed under the component's primary license
    If specified in the source, the code may be explicitly licensed under one of the component's alternate licenses
    I have the right to release the code I commit under the terms I am committing it

thank you

comment:3 Changed 5 years ago by zzz

Resolution: → fixed
Status: accepted → closed

tested here on two already-working UPnP devices
license agreement received from OP via email
in f4ed2082d62774111ba378ecc7161fb435cfcf7d 0.9.21-5-rc

Note: See TracTickets for help on using tickets.