Problem/Motivation

BEFORE:

$regex = '/\.(' . preg_replace('/ +/', '|', preg_quote($extensions)) . ')$/i';

AFTER:

$regex = '/\.(' . preg_replace('/ +/', '|', preg_quote($extensions, NULL)) . ')$/i';

Issue fork cloud-3241224

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

yas created an issue. See original summary.

yas’s picture

Assigned: Unassigned » yas
Status: Active » Needs review

@masami

Could you please review the patch? Thanks!

yas’s picture

@masami

Thank you for your review. I applied your suggestion, so could you please check the patch again? Thanks!

masami’s picture

Status: Needs review » Reviewed & tested by the community

@yas
It looks good to me now, so I change the status to RTBC.

yas’s picture

@masami

Thank you for your review. I'll merge the patch to 4.x and close this issue as Fixed.

  • yas committed 37d598b on 4.x
    Issue #3241224 by yas, Masami: Fix proper escaping by adding regex...

yas’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.