Ticket #54 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Remote SQL injections

Reported by: spam1@… Owned by: michiel
Priority: high Milestone: MvBlog 1.7
Component: global Version: 2.0
Severity: Keywords: vulnerability injection
Cc:

Description

MvBlog suffers from multiple remote SQL injections in the PHP code. Data gathered from the client-side is not verified and properly escaped before the SQL queries are built and executed. This can allow mallicious users (both log authors and remote users) to inject various different SQL code into the queries which in turn are executed.

Attacks, in typical SQL injection scenarios, allow attackers to delete or overwrite all data in the database, send out spam or gain escalated privileges in the system. One of the most visible and dangerous examples is in the comment posting code, but SQL injections can be found at various places.

Change History

Changed 3 years ago by michiel

  • status changed from new to assigned
  • version set to 2.0
  • milestone set to MvBlog 2.0

Thanks. I'm now walking through the code, putting preg_quote and sprintf's in place All other work is stopped till this is resolved.

Changed 3 years ago by spam1@…

You may want to put this ticket into non-disclosure mode, if that's possible, before advisory sites pick it up while there isn't a vendor provided patch available yet.

Changed 3 years ago by michiel

This has been adressed in the public interface.
The admin interface will be done tomorrow, but this is less critical (still critical, but can be fixed tomorrow) because an author has to login and do it by purpose. All the author can do with this is kill their own blog. Shouldn't be something happening, but we have to make sure it cannot happen

For the fix, see [105] [106] [107] [108] [109] [110] [111] [112]

See also ticket #55

Changed 3 years ago by michiel

Tickets cannot be set to non-disclosure mode :(

I'm posting a ticket on trac's website now.

Changed 3 years ago by michiel

(In [113]) sanity checks for the login screen

Re #54

Changed 3 years ago by michiel

(In [114]) Remember me to remove old code please

Re #54

Changed 3 years ago by michiel

(In [115]) Fix a lot of possible sql-injections.

Re #54

Changed 3 years ago by michiel

  • status changed from assigned to closed
  • resolution set to fixed

(In [118]) Fix last sql-injections we can find. Fixed some XSS attacks in the backend.

Fixes #54, Re #55

Changed 2 years ago by spam1@…

I've been receiving spam from MvBlog's Trac bugtracker, so I'm closing this email address (spam1@…). If you need to reach me you can find my email address at http://www.electricmonk.nl/index.php?page=Contact

Note: See TracTickets for help on using tickets.