Knowing how friendly to Xoopseres you are, I'm really disappointed with you this time, GIJOE. If you believe in you are capable of fixing the security hole, confident that wjue is unable to fix it, why don't you help wjue make the secure Agenda-X....
And I'm afraid it's quite natural for us to think GIJOE concealed the onokazu's post which solves this problem on purpose. Since I know you must have known the information on xoops.jp for the temporary fix when you started this thread....
You said wjue must take you for his foe. But, as for me, it seems to be you that take wjue for your foe...
Unfortunately, I'm not a skilled PHP coder, though I want to know the cause of this security hole. What was the problem? Is this security hole only related with global_register? What about open_basedir, or safe_mode?
You said Agenda-X would damage the value, and quality of Xoops. Moreover you stated pical is worth deserving the no.1 calendar module...
But what is XOOPS at all? XOOPS itself consists of thousands of ordinary users who know nothing about PHP. What we have to care is not the name of Xoops, but the community of XOOPS itself... Do you wana still keep developing such a top notch module even if nobody uses Xoops?
Well, what I want to say is that many people still use agenda-x, and they don't want to lose the data. However you suggested that they should remove agenda-x and use other alternatives, which results in abandoning the past data.
Of cource, considering its potential threat, it is the best way to remove the module, though...
==The files which need fixing==
- modules/agendax/addevent.inc.php
- modules/agendax/i18n.php
- modules/agendax/config.inc.php
- modules/agendax/admin/admin_header.php
- modules/agendax/addevent.inc.php
BEFORE
Quote:
include($agendax_path.\"/checkemail.inc.php\");
AFTER
Quote:
require_once './config.inc.php';
include($agendax_path.\"/checkemail.inc.php\");
- modules/agendax/i18n.php
BEFORE
Quote:
include_once $agendax_path.'/gettext.php';
AFTER
Quote:
require_once './config.inc.php';
include_once $agendax_path.'/gettext.php';
- modules/agendax/config.inc.php
BEFORE
Quote:
$agendax_path = XOOPS_ROOT_PATH.'/modules/agendax';
AFTER
Quote:
if (!defined('XOOPS_ROOT_PATH')) {
exit();
}
$agendax_path = XOOPS_ROOT_PATH.'/modules/agendax';
- modules/agendax/admin/admin_header.php
BEFORE
Quote:
if (!isset($agendax_path)) $agendax_path =XOOPS_ROOT_PATH.'/modules/agendax';
if (!isset($agendax_path)) $agendax_url = XOOPS_URL.'/modules/agendax';
AFTER
Quote:
require_once '../config.inc.php';