Skip to content

Conversation

@codengine
Copy link

I just noticed that when I tried to load images via Chrome on Windows 8 (as a non-administrator), the buffer for extracting the meta information was always unitizialized.

A proper additional check for the object and a instance of ArrayBuffer just ensured that the image could be read.

@blueimp
Copy link
Owner

blueimp commented Jan 29, 2014

Sorry but I could not reproduce this issue - just tried loading an JPEG image with EXIF information on the demo with Chrome on Windows 8 (virtual machine).
Could you provide the sample image with which you encountered this issue?
And are you also encountering this issue on the demo?

@codengine
Copy link
Author

Hey, thanks for your quick reply.
The problem exists with my project as well as with the demo inside a fresh copy of the newest version.

The test case is in fact hard to reproduce, it was:

  • Image residing on a NAS (shouldn't matter)
  • As well as a random local image
  • Windows 8.0
  • Chrome 32.0.1700.102 m (works fine at Firefox though)
  • User logged in as a non-privileged standard user (the Admin Account is password protected)

Running Chrome as an Administrator fixes the error in both cases, so there should be a problem between the non-privileged user, Windows 8, the FileSystem API and your class. Hard to track, but reproducable in our company.

Hope that helps.

Best Regards

@blueimp
Copy link
Owner

blueimp commented Jan 30, 2014

OK, this issue might just be the result of the missing FileReader error handling.
I've added it in the latest commit here:
000deb7
It's very close to your fix, but focuses on the error object itself.
Please let me know if this fixes the issue for you.

@codengine
Copy link
Author

I just tried the same thing as yesterday and it didn't occur anymore, with either version of your library. This is very strange and I just cant reproduce the circumstances. I even set up a Virtual Machine with the same setup as at our office. Anyway, a fail check for your meta-lib is always nice.

@blueimp
Copy link
Owner

blueimp commented Feb 5, 2014

Alright, thanks for the feedback. 👍

@blueimp blueimp closed this Feb 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants