Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

html4 + Win7 IE9 Access Denied Security Error #633

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

A security error was caused by the line "file: currentFile" in Win7 IE9. When the error happens in the catch block, it does not trigger the 'Error' handler. Add a finally without the file to ensure the error triggers the 'Error' handler.

Other users with similar issue and more info:
http://www.plupload.com/punbb/viewtopic.php?pid=394#p394
http://www.plupload.com/punbb/viewtopic.php?id=1793

@freak3dot freak3dot Update src/javascript/plupload.html4.js
A security error was caused by the line "file: currentFile" in Win7 IE9. When the error happens in the catch, it does not trigger the 'Error' handler. Add a finally without the file to ensure the error triggers the 'Error' handler.
26a001c
@freak3dot freak3dot Update src/javascript/plupload.html4.js
Error in logic. Finally executes no matter what. I just wanted to another catch in the previous catch block.
References #633
05ba07a
Owner

jayarjo commented Jan 26, 2013

Hm... this doesn't look as an elegant solution to the problem... Anyway we take this into account in Plupload 2.

@jayarjo jayarjo closed this Jan 28, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment