No input saniziting, no error handling on installation

Found a bug? Tell us!!
Post Reply
Zugschlus
Regular
Posts: 144
Joined: Sun Feb 05, 2006 12:54 am
Location: St. Ilgen, Germany
Contact:

No input saniziting, no error handling on installation

Post by Zugschlus »

Hi,

when I installed s9y yesterday, I chose "mh-s9y" as database user name and database and table name prefix.

s9y happily accepted these names (while it shouldn't have if "-" is a forbidden character in mySQL table names), and proceeded to issue SQL statements with the broken name. All of them were rejected and caused SQL errors, but s9y continued to issue SQL commands and finally said "success" (error handling either not present or badly broken).

Afterwards, s9y refused to enter configuration state again since it thought it was successfully configured. I had to empty serendipity_config_local.php.inc to be allowed to undo my mistake.

Greetings
Marc
--
Marc Haber, St. Ilgen, Germany
https://blog.zugschlus.de/ - nach langer Pause jetzt wieder online
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: No input saniziting, no error handling on installation

Post by garvinhicking »

Thanks, this is true. We'll work on it. :-)

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: No input saniziting, no error handling on installation

Post by garvinhicking »

Testing the DB prefix is technically a bit hard to do and will require temporary table creation. We'll see about this after the 1.0 release.

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Post Reply