strerror_r() should be used instead of strerror()
Bug #606478 reported by
Hartmut Holzgraefe
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Drizzle |
Fix Released
|
Medium
|
Andrew Hutchings | ||
| 7.0 |
Fix Released
|
Medium
|
Andrew Hutchings | ||
Bug Description
Several plugins use strerror() to get textual error messages.
strerror() uses an internal buffer though and so isn't necessarily thread safe.
The thread safe strerror_r() variant should be used instead.
Related branches
lp://staging/~linuxjedi/drizzle/drizzle-bug-606478
- Andrew Hutchings: Needs Resubmitting
- Brian Aker: Needs Fixing
-
Diff: 339 lines (+64/-19)11 files modifieddrizzled/definitions.h (+3/-0)
drizzled/errmsg_print.cc (+4/-2)
drizzled/message/transaction_reader.cc (+4/-1)
plugin/logging_gearman/logging_gearman.cc (+4/-1)
plugin/logging_query/logging_query.cc (+4/-1)
plugin/transaction_log/background_worker.cc (+5/-2)
plugin/transaction_log/hexdump_transaction_message.cc (+7/-2)
plugin/transaction_log/module.cc (+11/-3)
plugin/transaction_log/print_transaction_message.cc (+7/-2)
plugin/transaction_log/transaction_log.cc (+9/-3)
plugin/transaction_log/transaction_log_reader.cc (+6/-2)
| Changed in drizzle: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| Changed in drizzle: | |
| assignee: | nobody → Andrew Hutchings (thelinuxjedi) |
| Changed in drizzle: | |
| status: | Confirmed → In Progress |
| Changed in drizzle: | |
| status: | In Progress → Fix Committed |
| Changed in drizzle: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
