DragonFly BSD
DragonFly commits List (threaded) for 2010-08
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

git: aicasm: Correct logic bug in the undefined register bit access detection code.


From: Sascha Wildner <swildner@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Mon, 9 Aug 2010 18:50:37 -0700 (PDT)

commit 6affd6269fbf0381b1ad47b4594617c269b18b5a
Author: Sascha Wildner <saw@online.de>
Date:   Tue Aug 10 03:45:18 2010 +0200

    aicasm: Correct logic bug in the undefined register bit access detection code.
    
    Quoting FreeBSD's commit msg:
    
    The code in question verifies that all register write operations only change
    bits that are defined (in the register definition file) for that effected
    register.  The bug effectively disabled this checking.
    
    o Fix the check by testing the opcode against all supported read ("and" based)
      operands.
    
    o Add missing bit definitions to the aic7xxx and aic79xx register definition
      files so that the warning (treated as a fatal error) does not spuriously
      fire.
    
    Taken-from: FreeBSD
    
    [While I'm here, also add a missing free() and raise WARNS to 6.]

Summary of changes:
 sys/dev/disk/aic7xxx/aic79xx.reg          |    2 +-
 sys/dev/disk/aic7xxx/aic7xxx.reg          |    2 +-
 sys/dev/disk/aic7xxx/aicasm/Makefile      |    2 +-
 sys/dev/disk/aic7xxx/aicasm/aicasm.c      |    3 ++-
 sys/dev/disk/aic7xxx/aicasm/aicasm_gram.y |   15 ++++++++++-----
 sys/dev/disk/aic7xxx/aicasm/aicasm_scan.l |    6 ++----
 6 files changed, 17 insertions(+), 13 deletions(-)

http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/6affd6269fbf0381b1ad47b4594617c269b18b5a


-- 
DragonFly BSD source repository



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]