flexinodes textarea field in the 4.6 version of the module isn't checking the filter_access of the user. so a user with permissions to post flexinode content including textarea fields is able to run php-code through the php-filter.

patch attached.

flexinode head shouldn't be affected as there is used check_plain instead of check_output, however imho it should use check_markup() as it displays the filter form...

Comments

fago’s picture

Status: Active » Needs review
chx’s picture

Status: Needs review » Fixed

I commited a fix which is quite different from this patch, i added a validate_hook.

fago’s picture

Status: Fixed » Needs review
StatusFileSize
new2.41 KB

this patch fixes this issue by validation hook for flexinode head.
however the hook_access implementation needs to be updated to work with head.

fago’s picture

Status: Needs review » Fixed

already fixed in head, too.

Anonymous’s picture

Status: Fixed » Closed (fixed)