libnginx-mod-http-fancyindex is using outdated version with missing html escaping

Bug #1979917 reported by Stian Skjelstad
264
This bug affects 2 people
Affects Status Importance Assigned to Milestone
nginx (Ubuntu)
Confirmed
Medium
Thomas Ward

Bug Description

This patch has not made it into libnginx-mod-http-fancyindex from upstream yet.

https://github.com/aperezdc/ngx-fancyindex/commit/2e9555e7596f96ede95723169aa7c4f2e1cb42e5

This causes filenames with html-entities to be served as-is to the end-user, possible XSS if filenames are generated by user.

Alex Murray (alexmurray)
affects: ubuntu → nginx (Ubuntu)
information type: Private Security → Public Security
Revision history for this message
Thomas Ward (teward) wrote :

We (Debian) are also checking and merging to Kinetic - can you check if this is patched in Debian as well? So I can make sure its patched in Salsa as well as here in Ubuntu.

Changed in nginx (Ubuntu):
importance: Undecided → Medium
assignee: nobody → Thomas Ward (teward)
Revision history for this message
Thomas Ward (teward) wrote :

Also if you have a CVE ID for this please add it to the bug.

Revision history for this message
Stian Skjelstad (mywave) wrote :

I did not find any CVE reports when searching for "fancyindex", so I assume there is no report on it. I stumbled across this issue on a discussion online when a user complained about a file with &not in the filename didn't appear correctly on a public archive.

Revision history for this message
Thomas Ward (teward) wrote :

which Ubuntu releases are affected by this?

Revision history for this message
Steve Beattie (sbeattie) wrote :

This was fixed in debian in nginx 1.18.0-7, and thus was incorporated into ubuntu for kinetic in https://launchpad.net/ubuntu/+source/nginx/1.20.2-2ubuntu2.

The module has been incorporated into the nginx source package since at least trusty, so without further investigation, I would assume trusty through jammy are affected.

Revision history for this message
Steve Beattie (sbeattie) wrote :

(the fixing commit landed in the fancyindex module version 0.5.2 which was incorporated in 1.18.0-7)

Changed in nginx (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public Security information  
Everyone can see this security related information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.