Coverity SECURE_CODING - CID 12528 - src/plugin.cpp - in function: CompPlugin::load(const char *) - [VERY RISKY]. Using "sprintf" can cause a buffer overflow when done incorrectly. Because sprintf() assumes an arbitrarily long string, callers must be careful not to overflow the actual space of the destination. Use snprintf() instead, or correct precision specifiers.
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Triaged
|
Medium
|
Unassigned | ||
0.9.9 |
Won't Fix
|
Medium
|
Unassigned |
Bug Description
This bug is exported from the Coverity Integration Manager on Canonical's servers. For information on how this is done please see this website: https:/
CID: 12528
Checker: SECURE_CODING
Category: No category available
CWE definition: http://
File: /tmp/buildd/
Function: CompPlugin:
Code snippet:
466
467 if (char* home = getenv ("HOME"))
468 {
469 boost::
CID 12528 - SECURE_CODING
[VERY RISKY]. Using "sprintf" can cause a buffer overflow when done incorrectly. Because sprintf() assumes an arbitrarily long string, callers must be careful not to overflow the actual space of the destination. Use snprintf() instead, or correct precision specifiers.
470 sprintf (plugindir.get(), "%s/%s", home, HOME_PLUGINDIR);
471
472 if (loaderLoadPlugin (p.get(), plugindir.get(), name))
473 return p.release();
474 }
475
summary: |
- Coverity SECURE_CODING - CID 12528 + Coverity SECURE_CODING - CID 12528 - src/plugin.cpp - in function: + CompPlugin::load(const char *) - [VERY RISKY]. Using "sprintf" can cause + a buffer overflow when done incorrectly. Because sprintf() assumes an + arbitrarily long string, callers must be careful not to overflow the + actual space of the destination. Use snprintf() instead, or correct + precision specifiers. |
Changed in compiz: | |
milestone: | none → 0.9.10.0 |
Changed in compiz: | |
milestone: | 0.9.10.0 → 0.9.10.2 |
Changed in compiz: | |
milestone: | 0.9.10.2 → 0.9.11.0 |
Changed in compiz: | |
status: | New → Triaged |
milestone: | 0.9.11.0 → 0.9.12.1 |
Changed in compiz: | |
milestone: | 0.9.12.1 → 0.9.12.2 |
Source file with Coverity annotations.