The code in imageapi_image_scale() tries to divide by $image->info['width'] regardless of what width is. If $image->info['width'] is not set for whatever reason, this can cause the php warning: division by zero

Patch attached (against git checkout 6.x-1.x) to simply return FALSE, as there is not enough data to _scale().

CommentFileSizeAuthor
#1 division-by-zero-1284010-1.patch850 bytesjzacsh

Comments

jzacsh’s picture

jzacsh commented
StatusFileSize
new division-by-zero-1284010-1.patch850 bytes

Attached patch to avoid division by zero.

David Hernández’s picture

David Hernández commented

Hi there,

I'm not sure if the patch provided on #1 is a good solution. Instead of adding a line to avoid the function causing the error, will be better to find where the error is generated and fix it there. Why the image object doesn't have enough information?

Regards,
David.

David Hernández’s picture

David Hernández commented
Issue summary: View changes

Clarifying reason for bug's occurance.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Issue summary: View changes

See also #1527794: warning: Division by zero in /var/www/default/modules/imageapi/imageapi.module on line 221.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

In my case, I'm seeing the &$image parameter being FALSE.

Caller is imagefield/imagefield_file.inc at line 67.

I am also seeing in the log: "The selected image handling toolkit can not correctly process _image_close." This appears to be coming from imageapi_toolkit_invoke when $image->toolkit is empty (actually the image object is empty).