Opened 3 years ago

Closed 3 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

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 3 years ago.
patch for this ticket

Download all attachments as: .zip

Change History (4)

Changed 3 years ago by kay

patch for this ticket

comment:1 Changed 3 years ago by zzz

  • Component changed from router/general to router/transport
  • Milestone changed from undecided to 0.9.22
  • Owner set to zzz
  • Parent Tickets set to 481
  • Status changed from new to accepted

comment:2 Changed 3 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 3 years ago by zzz

  • Resolution set to fixed
  • Status changed from accepted to 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.