diff options
| author | Owen Taylor <otaylor@redhat.com> | 2003-01-28 23:25:47 +0000 |
|---|---|---|
| committer | Owen Taylor <otaylor@src.gnome.org> | 2003-01-28 23:25:47 +0000 |
| commit | cd535574bf46be12d94b83072b99af7365d58644 (patch) | |
| tree | 21a5a204db43823babad5c671d6d4e156348693f /pango | |
| parent | 33c2998d68a59af8116bd710f29121be6274bb8d (diff) | |
| download | pango-cd535574bf46be12d94b83072b99af7365d58644.tar.gz | |
Fix an off-by-one error when checking if are off the end of the coverage
Tue Jan 28 18:20:45 2003 Owen Taylor <otaylor@redhat.com>
* pango/pango-coverage.c (pango_coverage_set/get): Fix
an off-by-one error when checking if are off the end
of the coverage array. (#102284, Morten Welinder.)
* pango/pango-coverage.c (pango_coverage_get/set): Add
g_return_if_fail for index < 0.
Diffstat (limited to 'pango')
| -rw-r--r-- | pango/pango-coverage.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/pango/pango-coverage.c b/pango/pango-coverage.c index 52a7c958..e91fd3ef 100644 --- a/pango/pango-coverage.c +++ b/pango/pango-coverage.c @@ -169,10 +169,11 @@ pango_coverage_get (PangoCoverage *coverage, int block_index; g_return_val_if_fail (coverage != NULL, PANGO_COVERAGE_NONE); + g_return_val_if_fail (index < 0, PANGO_COVERAGE_NONE); block_index = index / 256; - if (block_index > coverage->n_blocks) + if (block_index >= coverage->n_blocks) return PANGO_COVERAGE_NONE; else { @@ -206,11 +207,12 @@ pango_coverage_set (PangoCoverage *coverage, guchar *data; g_return_if_fail (coverage != NULL); + g_return_if_fail (index < 0); g_return_if_fail (level >= 0 || level <= 3); block_index = index / 256; - if (block_index > coverage->n_blocks) + if (block_index >= coverage->n_blocks) { int old_n_blocks = coverage->n_blocks; |