Ticket Change Details
Not logged in
Tcl 2015 Conference, Manassas/VA, US, Oct 19-23
Send your abstracts to tclconference@googlegroups.com by Aug 24.

Artifact ID: 39659cb9da3fd3cb1c0d72cff679864c284da483
Ticket: 4b3b7a30821038c72de6cbb4f96aa20dd8345613
tcl8.5.15/generic/tclExecute.c:7713: array index before sanity check ?
Date: 2014-01-30 12:41:40
User: anonymous

  1. Change assignee to "nobody"
  2. Change closer to "nobody"
  3. Change cmimetype to "text/plain"
  4. Change comment 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.

  5. Change foundin to "8.5.15"
  6. Change is_private to "0"
  7. Change login to "anonymous"
  8. Change priority to "5 Medium"
  9. Change private_contact to "f533d94444fa1c1e17d8636ae60bdc0a92db0074"
  10. Change resolution to "None"
  11. Change severity to "Minor"
  12. Change status to "Open"
  13. Change submitter to "anonymous"
  14. Change subsystem to "47. Bytecode Compiler"
  15. Change title to:

    tcl8.5.15/generic/tclExecute.c:7713: array index before sanity check ?

  16. Change type to "Bug"