Go Back   vb.org Archive > vBulletin Modifications > Archive > vB.org Archives > Premium Archives > uCash & uShop
FAQ Community Calendar Today's Posts Search

Reply
 
Thread Tools
SQL Injection Issue Details »»
SQL Injection Issue
Version: , by radicaledward radicaledward is offline
Developer Last Online: Dec 2007 Show Printable Version Email this Page

Version: Unknown Rating:
Released: 03-12-2005 Last Update: Never Installs: 0
 
No support by the author.

Recently a member on my forums found a way to change the usertitles of all members of the site (about 10k) using an SQL injection exploit that they found in the "Change Other User's Custom Title" section of the shop, however, it is likely that it may be in other parts as well.

The basic way that the did it was a work around of the addslashes() that the script uses in the query. However, based on my reading (and testing) replacing addslashes() with mysql_real_escape_string() should do the trick for preventing it.

Show Your Support

  • This modification may not be copied, reproduced or published elsewhere without author's permission.

Comments
  #2  
Old 03-12-2005, 11:48 AM
Zachery's Avatar
Zachery Zachery is offline
 
Join Date: Jul 2002
Location: Ontario, Canada
Posts: 11,440
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by radicaledward
Recently a member on my forums found a way to change the usertitles of all members of the site (about 10k) using an SQL injection exploit that they found in the "Change Other User's Custom Title" section of the shop, however, it is likely that it may be in other parts as well.

The basic way that the did it was a work around of the addslashes() that the script uses in the query. However, based on my reading (and testing) replacing addslashes() with mysql_real_escape_string() should do the trick for preventing it.
What version of the hack are you using?
Reply With Quote
  #3  
Old 03-12-2005, 03:32 PM
Link14716's Avatar
Link14716 Link14716 is offline
 
Join Date: Jun 2002
Location: Georgia, USA
Posts: 2,519
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by radicaledward
Recently a member on my forums found a way to change the usertitles of all members of the site (about 10k) using an SQL injection exploit that they found in the "Change Other User's Custom Title" section of the shop, however, it is likely that it may be in other parts as well.

The basic way that the did it was a work around of the addslashes() that the script uses in the query. However, based on my reading (and testing) replacing addslashes() with mysql_real_escape_string() should do the trick for preventing it.
I'll definately look into it. I'm sure it could happen in 0.95, but it shouldn't in 0.95b.

I could see how it could happen if they manipulate the "userid" in the form, but that is it. Can I get more info on this?
Reply With Quote
  #4  
Old 03-13-2005, 05:27 AM
radicaledward's Avatar
radicaledward radicaledward is offline
 
Join Date: Jul 2004
Posts: 16
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

As far as I know the server is running 0.95b, I'm re-uploading the files just to be on the safe side. However, here is the text from one of the messages I got in regards to the issue:

Quote:
Quote:
Originally Posted by User message
Right, as you might have guessed, usertitles of every single ACF member were change to "LFF's +++++" twice and "Edman's +++++" to end it all up. No harm done, I was jsut having fun, and a bunch of people were having fun too. Now obviously, I did not spend 3 million gil and the tedious job of changing 10,000+ user titles.

There is an error in the store scripts. And it's bad. I'm not sure how much damage can be done to the forums using this, but I did not bother trying to find out. Basically put, the store's confirm page does not check if value is an INT type before passing it onto the SQL query.
Reply With Quote
  #5  
Old 03-13-2005, 11:55 PM
Link14716's Avatar
Link14716 Link14716 is offline
 
Join Date: Jun 2002
Location: Georgia, USA
Posts: 2,519
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by radicaledward
As far as I know the server is running 0.95b, I'm re-uploading the files just to be on the safe side. However, here is the text from one of the messages I got in regards to the issue:
Alright. I guess that means 0.95c is coming out tonight.
Reply With Quote
Reply


Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT. The time now is 06:17 AM.


Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.06065 seconds
  • Memory Usage 2,241KB
  • Queries Executed 20 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)ad_showthread_beforeqr
  • (5)bbcode_quote
  • (1)footer
  • (1)forumjump
  • (1)forumrules
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (1)modsystem_post
  • (1)navbar
  • (6)navbar_link
  • (120)option
  • (5)post_thanks_box
  • (5)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (5)post_thanks_postbit_info
  • (4)postbit
  • (5)postbit_onlinestatus
  • (5)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open
  • (1)tagbit_wrapper 

Phrase Groups Available:
  • global
  • inlinemod
  • postbit
  • posting
  • reputationlevel
  • showthread
Included Files:
  • ./showthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_postinfo_query
  • fetch_postinfo
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showthread_start
  • showthread_getinfo
  • forumjump
  • showthread_post_start
  • showthread_query_postids
  • showthread_query
  • bbcode_fetch_tags
  • bbcode_create
  • showthread_postbit_create
  • postbit_factory
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • tag_fetchbit_complete
  • forumrules
  • navbits
  • navbits_complete
  • showthread_complete