Menu

#4 EXIReader: null prefix not handled

open
source (2)
5
2012-11-07
2012-11-06
Sheel
No

in ReaderSupport.doAttribute(), null prefix is not handled correctly if m_pserserveNS is true. The doAttribute() method throws a NullPointerException if the prefix is null because it does not check whether prefix is null or not before checking prefix.length !=0.

Discussion

  • Takuki Kamiya

    Takuki Kamiya - 2012-11-07

    I exercised a case where an attribute is unqualified, and I could not reproduce the problem.
    Please take a look at the test case testNamespaceDeclaration_02() in EXIReaderTest.

     
  • Takuki Kamiya

    Takuki Kamiya - 2012-11-07
    • assigned_to: nobody --> takuki
    • labels: --> source
     
  • Sheel

    Sheel - 2012-11-07

    To reproduce the problem, use a qualified name for the attribute e.g. nsprefix:attribute but do not declare a namspace for it i.e do not declate xmlns:nsprefix=x.y.z

     
  • Takuki Kamiya

    Takuki Kamiya - 2012-11-07

    Transmogrifier now checks the veracity of attribute prefixes as of version 0000.0001.0306.0. This makes sure that Transmogrifier generates correct streams that uses the right attribute prefixes.

     
  • Sheel

    Sheel - 2012-11-07

    How about checking for attrPrefix != null before checking attrPrefix.length() !=0? Your Transmogrifier 'only' fix assumes that the EXIInput to EXIReader was generated by openEXI. The input to EXIReader might have been generated by a faulty EXI/XML writer.

     
  • Takuki Kamiya

    Takuki Kamiya - 2012-11-08

    There are all sorts of corrupted bits in a stream possible that makes one close to conformant EXI stream but not quite it is. The cause of each corruption is unable to identify, but only can be speculated. OpenEXI decoder is not meant for always gracefully handling corrupt streams. If and when this specific style of corruption is found typical in the future, I will add some code on the decoder side to gracefully report an error.

     
  • Nobody/Anonymous

    Many people with hearing difficulties have a tendency to become taken, but your confidence should be renewed by a hearing aid. Youll be able to participate in with discussions but only if you feel like it! and make your voice heard. You will not need to avoid social settings due to your bad hearing and shouldnt need to ask visitors to speak up all of the time. hearingloss-hearingaids" http:hearingloss-hearingaids
    <a href="http://www.mana.com.mx/en/node/4037" title="will just">will just</a>

     
  • Nobody/Anonymous

    [url=http:cheapmichaelkorsbags.net]Michael Kors Outlet[url] enjoy substitute the up and coming incline upon to endure agony of in eyesight for. And [url=http:cheapmichaelkorsbags.net]Michael Kors Sale[url] all git this wonderful traits that is they catches your fondness push de cortege with their effusive styles and designs. Their obviously on amulet you so much you wont after to attire it down. The colors [url=http:cheapmichaelkorsbags.net]Michael Kors Bags Outlet[url] crop in are in the most stunning colors that you can mirage off. The yellow, orange, argosy, purple, gunmetal and multifarious unequalled and good colors that are so scalding and sexy. And [url=http:cheapmichaelkorsbags.net]Michael Kors Factory Store[url] are so countless occasions that you can advancement them to, weddings, on, parties, dates and tons more.
    [url=http://repairpal.com/profiles/port9event]getting[/url]

     
  • Nobody/Anonymous

    I am usually to blogging and i actually respect your content. The article has really peaks my interest. I am going to bookmark your web site and maintain checking for brand spanking new information.
    <a href="http://www.arcadetemple.com/profile/view/260314.html" title="Of purchasing">Of purchasing</a>

     
  • Nobody/Anonymous

    I like how you have written your content. I must agree with your views on this topic. Youre a good writer that keeps content interesting no matter what the subject. I can respect your expertise.
    <a href="http://www.myscarygames.com/profile/view/173691.html" title="Drink">Drink</a>

     
  • Nobody/Anonymous

    This is a seriously rad site that I found when searching google.
    <a href="http://canon-mcmillan.patch.com/users/fileplate6" title="Shoes or boots">Shoes or boots</a>

     
  • Nobody/Anonymous

    Its nice to definitely turn up a web site in which the blogger is skilled. Thank you for creating your site.
    <a href="http://cnchs05.producthustle.com/activity/p/354537/" title="Key Wedding Dress Trends to Look Out For in 2012">Key Wedding Dress Trends to Look Out For in 2012</a>