annotate lib/wcswidth-impl.h @ 14418:07a5551df5ac

wcswidth, mbswidth: Avoid integer overflow. * lib/wcswidth.c: Include <limits.h>. * lib/wcswidth-impl.h (wcswidth): Avoid 'int' overflow. * lib/mbswidth.c: Include <limits.h>. (mbsnwidth): Avoid 'int' overflow. Reported by Jim Meyering.
author Bruno Haible <bruno@clisp.org>
date Sat, 12 Mar 2011 13:54:43 +0100
parents 7357862f4745
children 8250f2777afc
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
14303
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
1 /* Determine number of screen columns needed for a size-bounded wide string.
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
2 Copyright (C) 1999, 2011 Free Software Foundation, Inc.
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
3 Written by Bruno Haible <bruno@clisp.org>, 1999.
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
4
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
5 This program is free software: you can redistribute it and/or modify
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
6 it under the terms of the GNU General Public License as published by
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
7 the Free Software Foundation; either version 3 of the License, or
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
8 (at your option) any later version.
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
9
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
10 This program is distributed in the hope that it will be useful,
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
11 but WITHOUT ANY WARRANTY; without even the implied warranty of
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
12 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
13 GNU General Public License for more details.
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
14
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
15 You should have received a copy of the GNU General Public License
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
16 along with this program. If not, see <http://www.gnu.org/licenses/>. */
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
17
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
18 int
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
19 wcswidth (const wchar_t *s, size_t n)
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
20 {
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
21 int count = 0;
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
22 for (; n > 0; s++, n--)
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
23 {
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
24 wchar_t c = *s;
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
25 if (c == (wchar_t)'\0')
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
26 break;
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
27 {
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
28 int width = wcwidth (c);
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
29 if (width < 0)
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
30 goto found_nonprinting;
14418
07a5551df5ac wcswidth, mbswidth: Avoid integer overflow.
Bruno Haible <bruno@clisp.org>
parents: 14303
diff changeset
31 if (width > INT_MAX - count)
07a5551df5ac wcswidth, mbswidth: Avoid integer overflow.
Bruno Haible <bruno@clisp.org>
parents: 14303
diff changeset
32 goto overflow;
14303
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
33 count += width;
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
34 }
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
35 }
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
36 return count;
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
37
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
38 found_nonprinting:
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
39 return -1;
14418
07a5551df5ac wcswidth, mbswidth: Avoid integer overflow.
Bruno Haible <bruno@clisp.org>
parents: 14303
diff changeset
40
07a5551df5ac wcswidth, mbswidth: Avoid integer overflow.
Bruno Haible <bruno@clisp.org>
parents: 14303
diff changeset
41 overflow:
07a5551df5ac wcswidth, mbswidth: Avoid integer overflow.
Bruno Haible <bruno@clisp.org>
parents: 14303
diff changeset
42 return INT_MAX;
14303
7357862f4745 New module 'wcswidth'.
Bruno Haible <bruno@clisp.org>
parents:
diff changeset
43 }