Fatal error: Cannot read property 'length' of null

[gabrielmaldi]

I'm getting Fatal error: Cannot read property 'length' of null on images that I've optimized with TinyPNG (but aren't corrupt). Here's an example image:

ImageMagick's identify outputs myImage.png PNG 29x29 29x29+0+0 8-bit sRGB 28c 409B 0.000u 0:00.000 so the image appears to be valid, but imagemin has trouble with it.

I'm using grunt-contrib-imagemin v0.9.4.

Thanks!

[sindresorhus]

Does this still happen with 1.0.0?

[nleclerc]

I get the same error on 1.0.0.

[nleclerc]

Ah no, not same, it's property 'contents' of undefined.
Caused by a png file also.

[acasademont]

same here :(

[acasademont]

btw, it seems to be working on my Debian 7 environment, but 8 fails, weird, any hints?

[davidmaxwaterman]

It's probably a different situation, but I figured I'd add this here in case anyone else comes across it.

I had this error and through adding console.log()s to print the filename in the imagemin task (node_modules/grunt-contrib-imagemin/tasks/imagemin.js), I found that the error was produced when the task was given a directory rather than a file (ie app/images). This was due to lines in my Gruntfile :

        files: [
          { expand: true, cwd: '.', src: ['app/images/**'], dest: 'build/' },
          { expand: true, cwd: '.', src: ['app/css/images/**'], dest: 'build/' }
        ]

When I changes them to (note the extra /* on the src):

        files: [
          { expand: true, cwd: '.', src: ['app/images/**/*'], dest: 'build/' },
          { expand: true, cwd: '.', src: ['app/css/images/**/*'], dest: 'build/' }
        ]

all worked as before.

[vivekkodira1]

I was still encountering this error despite changing the path as above. Discovered that data[0].contents is returned as null sometimes. Changed the following line to get past it.

Before
var diffSize = origSize - data[0].contents.length;

After
var diffSize = origSize - ( ( data[0].contents && data[0].contents.length) || 0 );

[kevva]

Fixed in 92963f1.