Source-Changes archive

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

CVS commit: [netbsd-8] src/sys/dev/acpi



Module Name:    src
Committed By:   martin
Date:           Fri Aug 16 15:24:41 UTC 2019

Modified Files:
        src/sys/dev/acpi [netbsd-8]: acpi_ec.c

Log Message:
Pull up following revision(s) (requested by msaitoh in ticket #1337):

        sys/dev/acpi/acpi_ec.c: revision 1.76
        sys/dev/acpi/acpi_ec.c: revision 1.77

- Fix a bug that acpiec_space_handler() doesn't access more than 64bit
  correctly. Found by kUBSan on Thinkpad X220. acpiec0 accessed 128bits from
  address 0xa0. The error message was:
        UBSan: Undefined Behavior in ../../../../dev/acpi/acpi_ec.c:672:32, shift exponent 64 is too large for 64-bit type 'long unsigned int'
- KNF.
- Make the case that width < 8 behave as the same as before. Pointed out by
  Joerg.
- Change "switch" to "if" for simplify.


To generate a diff of this commit:
cvs rdiff -u -r1.75 -r1.75.6.1 src/sys/dev/acpi/acpi_ec.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.




Home | Main Index | Thread Index | Old Index