Uploading a picture in 1 bit Monochrome bmp generates logs in owncloud.log #27044

Closed
davitol opened this Issue Jan 27, 2017 · 16 comments

Comments

Projects
None yet
5 participants
@davitol
Contributor

davitol commented Jan 27, 2017

Steps to reproduce

  1. Upload a picture in 1 bit Monochrome bmp.
  2. Check owncloud.log file

Expected behaviour

No logs should be shown

Actual behaviour

Logs are written in owncloud.log

Server configuration

Operating system:
Ubuntu 16.04

Web server:
Apache

Database:
MySQL

PHP version:
5.6

ownCloud version:

"version":"8.1.11"

Updated from an older ownCloud or fresh install:
Fresh

Are you using external storage, if yes which one: local/smb/sftp/...
No

Are you using encryption:
No

Logs

{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}
{"reqId":"J4MDdcQ32aMOaOjxxemJ","remoteAddr":"82.159.139.58","app":"PHP","message":"Undefined offset: 2 at \/opt\/owncloud\/lib\/private\/image.php#773","level":3,"time":"2017-01-27T12:03:23+00:00"}

Client configuration

browser

Mozilla

@davitol davitol added this to the 8.1.12 milestone Jan 27, 2017

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Jan 27, 2017

"version":"8.1.11"

Did you tried the yesterdays released 8.1.2 RC2 https://central.owncloud.org/t/owncloud-server-maintenance-releases-9-1-4-rc2-9-0-8-rc2-8-2-10-rc2-8-1-12-rc2/5315

Might be possible that the fix of #26849 is already fixing this.

ghost commented Jan 27, 2017

"version":"8.1.11"

Did you tried the yesterdays released 8.1.2 RC2 https://central.owncloud.org/t/owncloud-server-maintenance-releases-9-1-4-rc2-9-0-8-rc2-8-2-10-rc2-8-1-12-rc2/5315

Might be possible that the fix of #26849 is already fixing this.

@davitol

This comment has been minimized.

Show comment
Hide comment
@davitol

davitol Jan 27, 2017

Contributor

@kdslkdsaldsal Yes, I did. In fact i was trying to check if that issue was solved, but I found this other behaviour. Not sure if @Temtaime did check if the RC2 fixes that issue or at least can provide a file as the one was located in the link of the issue #26849

Contributor

davitol commented Jan 27, 2017

@kdslkdsaldsal Yes, I did. In fact i was trying to check if that issue was solved, but I found this other behaviour. Not sure if @Temtaime did check if the RC2 fixes that issue or at least can provide a file as the one was located in the link of the issue #26849

@PVince81

This comment has been minimized.

Show comment
Hide comment
@PVince81

PVince81 Jan 27, 2017

Member

@VicDeo please have a look, might need more ampersands...

Member

PVince81 commented Jan 27, 2017

@VicDeo please have a look, might need more ampersands...

@VicDeo

This comment has been minimized.

Show comment
Hide comment
@VicDeo

VicDeo Jan 27, 2017

Member

@PVince81 is milestone correct?
IIRC branches are locked for all PRs except showstoppers and 8.1 reaches EOL after this release

Member

VicDeo commented Jan 27, 2017

@PVince81 is milestone correct?
IIRC branches are locked for all PRs except showstoppers and 8.1 reaches EOL after this release

@PVince81 PVince81 added sev2-high and removed sev4-low labels Jan 27, 2017

@PVince81

This comment has been minimized.

Show comment
Hide comment
@PVince81

PVince81 Jan 27, 2017

Member

Setting to sev high, log pollution triggered by users sucks.

Member

PVince81 commented Jan 27, 2017

Setting to sev high, log pollution triggered by users sucks.

@PVince81 PVince81 modified the milestones: 10.0, 8.1.12 Jan 27, 2017

@VicDeo

This comment has been minimized.

Show comment
Hide comment
@VicDeo

VicDeo Jan 27, 2017

Member

@PVince81 I this if the color is out of palette range it's better to break immediately and do not waste resources trying to build preview for potentially FUBAR data.

Member

VicDeo commented Jan 27, 2017

@PVince81 I this if the color is out of palette range it's better to break immediately and do not waste resources trying to build preview for potentially FUBAR data.

@VicDeo

This comment has been minimized.

Show comment
Hide comment
@VicDeo

VicDeo Jan 27, 2017

Member

@davitol which bmp did you try?

Member

VicDeo commented Jan 27, 2017

@davitol which bmp did you try?

@davitol

This comment has been minimized.

Show comment
Hide comment
@davitol

davitol Jan 30, 2017

Contributor

@davitol which bmp did you try?

@VicDeo shared with you privately

Contributor

davitol commented Jan 30, 2017

@davitol which bmp did you try?

@VicDeo shared with you privately

@SergioBertolinSG

This comment has been minimized.

Show comment
Hide comment
@SergioBertolinSG

SergioBertolinSG Jan 31, 2017

Member

This is also happening in 9.1.4RC2.

Member

SergioBertolinSG commented Jan 31, 2017

This is also happening in 9.1.4RC2.

@PVince81

This comment has been minimized.

Show comment
Hide comment
@PVince81

PVince81 Jan 31, 2017

Member

@VicDeo any update / quick fix ?

Member

PVince81 commented Jan 31, 2017

@VicDeo any update / quick fix ?

@davitol

This comment has been minimized.

Show comment
Hide comment
@davitol

davitol Jan 31, 2017

Contributor

@PVince81 IMHO the severity should be lower, only a few lines are written due to this action. Not really a flood problem

Contributor

davitol commented Jan 31, 2017

@PVince81 IMHO the severity should be lower, only a few lines are written due to this action. Not really a flood problem

@PVince81 PVince81 added sev3-medium and removed sev2-high labels Jan 31, 2017

@PVince81

This comment has been minimized.

Show comment
Hide comment
@PVince81

PVince81 Jan 31, 2017

Member

Done. Also I think an admin can disable preview for BMP files in config.php

Member

PVince81 commented Jan 31, 2017

Done. Also I think an admin can disable preview for BMP files in config.php

@VicDeo

This comment has been minimized.

Show comment
Hide comment
@VicDeo

VicDeo Jan 31, 2017

Member

@PVince81 I think this is not for rc2 phase, but anyway
Stable9.1: #27065
Stable9: #27066
Stable8.2: #27067

Member

VicDeo commented Jan 31, 2017

@PVince81 I think this is not for rc2 phase, but anyway
Stable9.1: #27065
Stable9: #27066
Stable8.2: #27067

@VicDeo

This comment has been minimized.

Show comment
Hide comment
@VicDeo

VicDeo Jan 31, 2017

Member

@davitol Could you confirm that it is not reproducible on master?

Member

VicDeo commented Jan 31, 2017

@davitol Could you confirm that it is not reproducible on master?

@davitol

This comment has been minimized.

Show comment
Hide comment
@davitol

davitol Feb 1, 2017

Contributor

@davitol Could you confirm that it is not reproducible on master?

@VicDeo Just tested and it is not reproducible on master

Contributor

davitol commented Feb 1, 2017

@davitol Could you confirm that it is not reproducible on master?

@VicDeo Just tested and it is not reproducible on master

@VicDeo

This comment has been minimized.

Show comment
Hide comment
@VicDeo

VicDeo Feb 6, 2017

Member

Closing as fixed

Member

VicDeo commented Feb 6, 2017

Closing as fixed

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