Unserialize untrusted data when importing skins

[Son Nguyen]

Version: 1.10, 15.04. 15.10, master
Platform: any

There is a unserialize vulnerability in skin import function

see line 200 in htdocs/skin/import.php

When importing the attached skin, you will see the error:

[WAR] ce (lib/web.php:3684) Object of class __PHP_Incomplete_Class could not be converted to string
Call stack (most recent first):
log_message("Object of class __PHP_Incomplete_Class could not b...", 8, true, true, "/var/www/mahara/master/htdocs/lib/web.php", 3684) at /var/www/mahara/master/htdocs/lib/errors.php:441
error(4096, "Object of class __PHP_Incomplete_Class could not b...", "/var/www/mahara/master/htdocs/lib/web.php", 3684, array(size 5)) at /var/www/mahara/master/htdocs/lib/web.php:3684
clean_css(object(__PHP_Incomplete_Class), true) at /var/www/mahara/master/htdocs/skin/import.php:200
importskinform_submit(object(Pieform), array(size 4)) at Unknown:0
call_user_func_array("importskinform_submit", array(size 2)) at /var/www/mahara/master/htdocs/lib/pieforms/pieform.php:537
Pieform->__construct(array(size 4)) at /var/www/mahara/master/htdocs/lib/pieforms/pieform.php:164
Pieform::process(array(size 4)) at /var/www/mahara/master/htdocs/lib/pieforms/pieform.php:71
pieform(array(size 4)) at /var/www/mahara/master/htdocs/skin/import.php:64

[Robert Lyon (robertl-9)]

https://reviews.mahara.org/#/c/5579/

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6677
Committed: https://git.mahara.org/mahara/mahara/commit/1f299954f3ffbc26c69e27f000daf8f0e97de457
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.04_STABLE

commit 1f299954f3ffbc26c69e27f000daf8f0e97de457
Author: Son Nguyen <email address hidden>
Date: Thu Oct 22 10:55:40 2015 +1300

Make sure imported custom skin xml entries are clean. Bug 1508684

behatnotneeded

Change-Id: I2e597d5931391e731baefa46d5f9d9ca2059ee10

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6679
Committed: https://git.mahara.org/mahara/mahara/commit/9d7701e80b24bdbaccb77ae7730ae9c504d1143b
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.04_STABLE

commit 9d7701e80b24bdbaccb77ae7730ae9c504d1143b
Author: Son Nguyen <email address hidden>
Date: Thu Oct 22 10:55:40 2015 +1300

Make sure imported custom skin xml entries are clean. Bug 1508684

behatnotneeded

Change-Id: I2e597d5931391e731baefa46d5f9d9ca2059ee10

[Mahara Bot (dev-mahara)]

Reviewed: https://reviews.mahara.org/6678
Committed: https://git.mahara.org/mahara/mahara/commit/3f9514bdaa9b70457a404cd1b9aa502c261aeef2
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.10_STABLE

commit 3f9514bdaa9b70457a404cd1b9aa502c261aeef2
Author: Son Nguyen <email address hidden>
Date: Thu Oct 22 10:55:40 2015 +1300

Make sure imported custom skin xml entries are clean. Bug 1508684

behatnotneeded

Change-Id: I2e597d5931391e731baefa46d5f9d9ca2059ee10