]> sourceware.org Git - systemtap.git/commit
Force correct order of evaluation of macro arguments in check_*register macros
authorWilliam Cohen <wcohen@redhat.com>
Thu, 2 May 2019 14:41:59 +0000 (10:41 -0400)
committerWilliam Cohen <wcohen@redhat.com>
Thu, 2 May 2019 14:41:59 +0000 (10:41 -0400)
commit446e7c3582475751136722e633085479fcb7dd31
tree2d36a330709d520cf83f2e5ead72c59c30eae644
parentc320e413c4b4f2c6d5043d6ec6ebf8f93afd834e
Force correct order of evaluation of macro arguments in check_*register macros

Noted that a number of tests were failing on x86 machines with errors
like the following:

ERROR: register access fault [man error::fault] near identifier 'module_name' at
 /usr/share/systemtap/tapset/linux/context.stp:392:10

The problem was traced to the maxregno argument for the macro having a
?: operator which has lower precedence than || or >.  This caused the
conditional tests in check_fetch_register and check_store_register for
error reporting to incorrectly trigger.  Used ()'s in the conditionals
to force the correct order of evaluation.
runtime/linux/loc2c-runtime.h
This page took 0.027854 seconds and 5 git commands to generate.