Issue 80067 - warning: comparison with string literal in binfilters
Summary: warning: comparison with string literal in binfilters
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: 680m222
Hardware: PC Linux, all
: P3 Trivial (vote)
Target Milestone: OOo 2.3
Assignee: christian.guenther
QA Contact: issues@framework
Keywords: regression
Depends on:
Blocks: 80294
  Show dependency tree
Reported: 2007-07-26 12:22 UTC by pmladek
Modified: 2007-09-20 10:40 UTC (History)
3 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---

Proposed fix. (622 bytes, text/plain)
2007-07-26 12:23 UTC, pmladek
no flags Details
Impress 5.0 sample document (986.50 KB, application/octet-stream)
2007-08-21 14:01 UTC, Mathias_Bauer
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description pmladek 2007-07-26 12:22:19 UTC
I have got the following warnings in the binfilter module in src680_m222:

--- cut ---
warning: comparison with string literal results in unspecified behaviour
warning: comparison with string literal results in unspecified behaviour
--- cut ---

The code is really broken. I'll attach a possible fix. I am not sure if the test
for NULL pointer is needed but it seems so...
Comment 1 pmladek 2007-07-26 12:23:38 UTC
Created attachment 47105 [details]
Proposed fix.
Comment 2 Olaf Felka 2007-07-26 15:28:06 UTC
@ as: Something from your playground?
Comment 3 Mathias_Bauer 2007-08-20 09:01:18 UTC
The NULL pointer test isn't necessary but it doesn't hurt. :-)
Thanks for the patch. As this is a regression I will try to get it fixed in 2.3.
Comment 4 Mathias_Bauer 2007-08-21 14:01:06 UTC
Created attachment 47678 [details]
Impress 5.0 sample document
Comment 5 Mathias_Bauer 2007-08-21 14:01:27 UTC
As the wrong string comparison damaged an old fix for an old problem I attached
the original sample document that now is imported wrongly.
Load the attached document and go to the slide "architektur". You will recognize
at least one wrongly positioned element immediately. The fix should correct that.
Comment 6 Mathias_Bauer 2007-08-21 14:04:54 UTC
added keyword "regression"
Comment 7 Mathias_Bauer 2007-08-21 14:05:17 UTC
please verify
Comment 8 christian.guenther 2007-08-21 15:24:23 UTC
CGU: Verified in cws binfilterregression.
Comment 9 christian.guenther 2007-09-20 10:40:49 UTC
CGU: Integrated in src680m228 and oog680m5