Modify

Opened 8 years ago

Closed 3 months ago

#2702 closed defect (fixed)

path is leaking some sensitive infos

Reported by: anonymous Owned by: cboos
Priority: normal Component: DoxygenPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description (last modified by rjollos)

Hi,

I'm not sure what's the ?paht=/full/path/diclosure/bla.html good for.

In my opinion it's disclosing potentialy sensitive informations.

Cheers, /thorkill

--- simple fix

  • 0.10/doxygentrac/doxygentrac.py

     
    237237                              href=formatter.href.doxygen())
    238238            else:
    239239                return html.a(label, title=params,
    240                               href=formatter.href.doxygen(link, path=path))
     240                              href=formatter.href.doxygen(link))
    241241        yield ('doxygen', doxygen_link)
    242242
    243243    def get_wiki_syntax(self):

Attachments (0)

Change History (3)

comment:2 Changed 6 years ago by anonymous

Any plans to merge this in SVN?

comment:3 Changed 6 years ago by rjollos

  • Description modified (diff)

comment:4 Changed 3 months ago by Committo-Ergo-Sum

  • Resolution set to fixed
  • Status changed from new to closed

In 15358:

DoxygenPlugin: new implementation of IRequestHandler methods. In particular, the "path" parameter in the query-string is droped. This fixes #772 #951 #962 #1564 and #2702 who complain about the security vulnerability it may contain, and other wrong path generations.

Add Comment

Modify Ticket

Action
as closed The owner will remain cboos.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.