Merge lp:~mc-return/compiz/compiz.merge-fix1101641-use-snprintf-instead-of-sprintf into lp:compiz/0.9.9
Proposed by
MC Return
Status: | Merged | ||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Approved by: | Timo Jyrinki | ||||||||||||||||||||||||
Approved revision: | 3593 | ||||||||||||||||||||||||
Merged at revision: | 3588 | ||||||||||||||||||||||||
Proposed branch: | lp:~mc-return/compiz/compiz.merge-fix1101641-use-snprintf-instead-of-sprintf | ||||||||||||||||||||||||
Merge into: | lp:compiz/0.9.9 | ||||||||||||||||||||||||
Diff against target: |
133 lines (+12/-12) 7 files modified
libdecoration/decoration.c (+1/-1) plugins/composite/src/screen.cpp (+1/-1) plugins/dbus/src/dbus.cpp (+2/-2) plugins/loginout/src/loginout.cpp (+2/-2) plugins/screenshot/src/screenshot.cpp (+1/-1) plugins/water/src/water.cpp (+3/-3) src/screen.cpp (+2/-2) |
||||||||||||||||||||||||
To merge this branch: | bzr merge lp:~mc-return/compiz/compiz.merge-fix1101641-use-snprintf-instead-of-sprintf | ||||||||||||||||||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sam Spilsbury | Approve | ||
PS Jenkins bot | continuous-integration | Pending | |
Review via email: mp+145228@code.launchpad.net |
Description of the change
As requested I am fixing all of those closely related SECURE_CODING issues
in just one MP, hope the bots can cope with that... ;)
To post a comment you must log in.
This is all good as is.
Generally stuff like this:
44 + snprintf (path, 256, "%s/%s/%s/%s", COMPIZ_ DBUS_ROOT_ PATH,
45 plugin.c_str (), "options", o->name ().c_str ());
scares me because it can be truncated quite easily (leading to subtle problems). But, it isn't a problem with this review generally and we can fix it later.