Opened 8 years ago

Closed 8 years ago

#7358 closed (invalid)

[patch] create_default_site: configurable default domain name

Reported by: erob Owned by: nobody
Component: Contrib apps Version: master
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:



The create_default_site function in the
module hardcodes as the default site. I think this is potentially
harmful for users because it can be exploited remotely.

A better hardcoded value would be to use 'localhost' or socket.gethostname
as the default domain name. The attached patch implements the later for resolving
the default domain name.

Attachments (1)

003_create_default_site.patch (1.3 KB) - added by erob 8 years ago.
Uses socket.gethostname to resolve the default domain name

Download all attachments as: .zip

Change History (2)

Changed 8 years ago by erob

Uses socket.gethostname to resolve the default domain name

comment:1 Changed 8 years ago by mtredinnick

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to invalid
  • Status changed from new to closed

Since is explicitly reserved in RFC 2606, it's not open to any kind of effective exploitation.

In future, perhaps you could have researched this rather than calling the developers stupid in the patch. Just a thought.

Note: See TracTickets for help on using tickets.
Back to Top