Overview
Artifact ID: | 39659cb9da3fd3cb1c0d72cff679864c284da483 |
---|---|
Ticket: | 4b3b7a30821038c72de6cbb4f96aa20dd8345613
tcl8.5.15/generic/tclExecute.c:7713: array index before sanity check ? |
User & Date: | anonymous 2014-01-30 12:41:40 |
Changes
- assignee changed to: "nobody"
- closer changed to: "nobody"
- cmimetype changed to: "text/plain"
- comment changed to:
I just ran the static analysis tool "cppcheck" over the source code of tcl8.5.15 It said many things, including [/home/dcb/rpmbuild/BUILD/tcl8.5.15/generic/tclExecute.c:7713] -> [/home/dcb/rpmbuild/BUILD/tcl8.5.15/generic/tclExecute.c:7715]: (warning) Array 'operatorStrings[34]' accessed at index 59, which is out of bounds. Otherwise condition 'opcode==99' is redundant. Source code is const char *description, *operator = operatorStrings[opcode - INST_LOR]; if (opcode == INST_EXPON) { operator = "**"; } Suggest move sanity check of opcode to before its use.
- foundin changed to: "8.5.15"
- is_private changed to: "0"
- login: "anonymous"
- priority changed to: "5 Medium"
- private_contact changed to: "f533d94444fa1c1e17d8636ae60bdc0a92db0074"
- resolution changed to: "None"
- severity changed to: "Minor"
- status changed to: "Open"
- submitter changed to: "anonymous"
- subsystem changed to: "47. Bytecode Compiler"
- title changed to:
tcl8.5.15/generic/tclExecute.c:7713: array index before sanity check ?
- type changed to: "Bug"