diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-08-07 12:32:45 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-08-07 18:00:44 +0100 |
| commit | 93a37a649755034d306767880572a5d81a43eceb (patch) | |
| tree | 07aa5e6d9fcb683392b7435c73400e5bb76e9c52 | |
| parent | f7f943d71faa8117f902289f95b9ffc7582a5320 (diff) | |
coverity#1224982 Out-of-bounds access
Change-Id: I028148a1112cdbe83ff7fb439cc4280a7723338b
| -rw-r--r-- | hwpfilter/source/hbox.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/hwpfilter/source/hbox.cxx b/hwpfilter/source/hbox.cxx index b22e9cf90ff4..4fd3b4d2b9da 100644 --- a/hwpfilter/source/hbox.cxx +++ b/hwpfilter/source/hbox.cxx @@ -189,7 +189,7 @@ hchar_string DateCode::GetString() cbuf[4] = 0; break; case '*': - strncat(cbuf, en_mon[date[MONTH] - 1], 255); + strncat(cbuf, en_mon[date[MONTH] - 1], sizeof(cbuf) - strlen(cbuf) - 1); break; case '3': /* 'D' is day of korean */ num = date[DAY]; @@ -231,17 +231,17 @@ hchar_string DateCode::GetString() cbuf[4] = 0; break; case '_': - strncat(cbuf, en_week[date[WEEK]], 256); + strncat(cbuf, en_week[date[WEEK]], sizeof(cbuf) - strlen(cbuf) - 1); break; case '7': ret.push_back(0xB5A1); ret.push_back((is_pm) ? 0xD281 : 0xB8E5); break; case '&': - strncat(cbuf, (is_pm) ? "p.m." : "a.m.", 256); + strncat(cbuf, (is_pm) ? "p.m." : "a.m.", sizeof(cbuf) - strlen(cbuf) - 1); break; case '+': - strncat(cbuf, (is_pm) ? "P.M." : "A.M.", 256); + strncat(cbuf, (is_pm) ? "P.M." : "A.M.", sizeof(cbuf) - strlen(cbuf) - 1); break; case '8': // 2.5 feature case '9': |