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

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


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

commit 0a99709aa9eb8d06ccdfd63f8b893e9cfc7a542b
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/0a99709aa9eb8d06ccdfd63f8b893e9cfc7a542b


-- 
DragonFly BSD source repository



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