[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
CVS: cvs.openbsd.org: src
- To: source-changes_(_at_)_cvs_(_dot_)_openbsd_(_dot_)_org
- Subject: CVS: cvs.openbsd.org: src
- From: Gordon Willem Klok <gwk_(_at_)_cvs_(_dot_)_openbsd_(_dot_)_org>
- Date: Sun, 28 Jan 2007 13:28:51 -0700 (MST)
Module name: src
Changes by: gwk_(_at_)_cvs_(_dot_)_openbsd_(_dot_)_org 2007/01/28 13:28:50
sys/dev/vesa : vesabiosvar.h vesabios.c vesafb.c
sys/dev/pci : vga_pci.c
Fix a number of issues with vesabios discovered by miod@
1. Add bounds checking to vga_pci_mmap, fixing a potential security issue.
Limit the amount of memory to what the vbe info structure says the card
card has. This should be nicely refined to limit to the amount of memory
needed by the active resolution and depth but this will do for the time
2. Fix several places in vesafb.c where the return of kvm86_bios_addpage()
isnt being checked. Also ensure that if vesafb_get_mode_info() fails that it
cleans up after itself by releasing the page it added.
3. Correct the range checks in vesafb_putcmap and vesafb_getcmap, harmonize
code with similar code found else where.