diff options
author | Tatsuya Kinoshita <tats@debian.org> | 2016-11-14 12:01:08 +0000 |
---|---|---|
committer | Tatsuya Kinoshita <tats@debian.org> | 2016-11-19 12:25:59 +0000 |
commit | 500e08adcd54bc033a97710aadc844bcea143be1 (patch) | |
tree | 087a1b7b70c7c4430eb324c45dc09a1f24a1ef60 | |
parent | Check indent_level to prevent infinite recursion (diff) | |
download | w3m-500e08adcd54bc033a97710aadc844bcea143be1.tar.gz w3m-500e08adcd54bc033a97710aadc844bcea143be1.zip |
Prevent array index out of bounds for symbol
Bug-Debian: https://github.com/tats/w3m/issues/38
Origin: https://anonscm.debian.org/cgit/collab-maint/w3m.git/commit/?id=0c3f5d0e0d9269ad47b8f4b061d7818993913189
-rw-r--r-- | display.c | 8 | ||||
-rw-r--r-- | file.c | 2 | ||||
-rw-r--r-- | fm.h | 1 | ||||
-rw-r--r-- | symbol.c | 4 |
4 files changed, 8 insertions, 7 deletions
@@ -1119,18 +1119,18 @@ addChar(char c, Lineprop mode) } #ifdef USE_M17N if (w == 2 && WcOption.use_wide) - addstr(graph2_symbol[(int)c]); + addstr(graph2_symbol[(unsigned char)c % N_GRAPH_SYMBOL]); else #endif - addch(*graph_symbol[(int)c]); + addch(*graph_symbol[(unsigned char)c % N_GRAPH_SYMBOL]); } else { #ifdef USE_M17N symbol = get_symbol(DisplayCharset, &w); - addstr(symbol[(int)c]); + addstr(symbol[(unsigned char)c % N_SYMBOL]); #else symbol = get_symbol(); - addch(*symbol[(int)c]); + addch(*symbol[(unsigned char)c % N_SYMBOL]); #endif } } @@ -7583,7 +7583,7 @@ conv_symbol(Line *l) symbol = get_symbol(DisplayCharset, &w); #endif } - Strcat_charp(tmp, symbol[(int)c]); + Strcat_charp(tmp, symbol[(unsigned char)c % N_SYMBOL]); #ifdef USE_M17N p += len - 1; pr += len - 1; @@ -1099,6 +1099,7 @@ extern char *graph2_symbol[]; extern int symbol_width; extern int symbol_width0; #define N_GRAPH_SYMBOL 32 +#define N_SYMBOL (N_GRAPH_SYMBOL + 14) #define SYMBOL_BASE 0x20 global int no_rc_dir init(FALSE); global char *rc_dir init(NULL); @@ -176,10 +176,10 @@ push_symbol(Str str, char symbol, int width, int n) #ifdef USE_M17N if (width == 2) - p = alt2_symbol[(int)symbol]; + p = alt2_symbol[(unsigned char)symbol % N_SYMBOL]; else #endif - p = alt_symbol[(int)symbol]; + p = alt_symbol[(unsigned char)symbol % N_SYMBOL]; for (i = 0; i < 2 && *p; i++, p++) buf[i] = (*p == ' ') ? NBSP_CODE : *p; |