Bug 145519 – www-apps/xoops SQL injection

• Status: RESOLVED INVALID
• Product: Gentoo Security

[Description Sune Kloppenborg Jeppesen (RETIRED) 2006-08-29 12:32:14 UTC]

There is a sql injection in Xoops 2.0.14 (and maybe before versions) .
 One of the user inputs, is used in the sql query without proper checking :
 
 File /edituser.php, Line 347 :
 :: if (!empty($_POST['user_avatar'])) {
 >> $user_avatar = trim($_POST['user_avatar']);
 :: $criteria_avatar = new CriteriaCompo(new Criteria('avatar_file', $user_avatar));
 :: $criteria_avatar->add(new Criteria('avatar_type', "S"));
 :: $avatars =& $avt_handler->getObjects($criteria_avatar);
 :: if (!is_array($avatars) || !count($avatars)) {
 :: $user_avatar = 'blank.gif';
 :: }
 
 The bug can be critical, so no more info .
 
 You can upgrade to 2.0.15 .
 Also, a simple solution is to change line 348 of /edituser.php, to :
 $user_avatar = addslashes(trim($_POST['user_avatar']));

[Comment 1 Thierry Carrez (RETIRED) 2006-09-02 01:53:57 UTC]

This is fixed in 2.0.15 (latest version on upstream site) but portage has 2.2.4 ??

[Comment 2 Renat Lumpau (RETIRED) 2006-09-02 05:03:10 UTC]

I believe 2.2.x is their dev branch. We currently don't have any ebuilds for the 2.0.x branch.

[Comment 3 Sune Kloppenborg Jeppesen (RETIRED) 2006-09-02 06:59:23 UTC]

Closing this one as INVALID then.