Fix memory corruption while configuring CPU windows on Marvell SoCs
ClosedPublic
Actions

Authored by mw_semihalf.com on May 14 2017, 12:58 AM.

Tags

Referenced Files

	Unknown Object (File)
	Tue, Dec 31, 8:16 AM

	Unknown Object (File)
	Mon, Dec 30, 2:42 AM

	Unknown Object (File)
	Mon, Dec 30, 1:57 AM

	Unknown Object (File)
	Sat, Dec 28, 4:05 PM

	Unknown Object (File)
	Fri, Dec 20, 8:18 PM

	Unknown Object (File)
	Fri, Dec 20, 4:11 PM

	Unknown Object (File)
	Thu, Dec 12, 7:39 PM

	Unknown Object (File)
	Fri, Dec 6, 4:59 AM

Subscribers

Details

Reviewers

zbb
loos
mmel
• ian

Commits

rS318874: Fix memory corruption while configuring CPU windows on Marvell SoCs

Summary

Resolving CPU windows from localbus entry caused buffer overflow
and memory corruption. Fix wrong indexing and ensure the index
does not exceed table size.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

mw_semihalf.com created this revision.May 14 2017, 12:58 AM

Herald added subscribers: andrew, imp. · View Herald TranscriptMay 14 2017, 12:58 AM

mw_semihalf.com added a parent revision: D10719: Enable proper parsing of nested simlpe-buses on Marvell platforms.May 14 2017, 12:58 AM

mw_semihalf.com added a child revision: D10721: Fix USB3.0 decoding windows on Armada38x.

I think it is easier to use nitems() here.

sys/arm/mv/mv_common.c
2158	if (tuples > nitems(cpu_win_tbl)) {
2196	if (t >= nitems(cpu_win_tbl)) {

Switched to nitems, thanks for the suggestion.

Closed by commit rS318874: Fix memory corruption while configuring CPU windows on Marvell SoCs (authored by zbb). · Explain WhyMay 25 2017, 2:17 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

arm/

mv/

mv_common.c

12 lines

Diff 28320

View Options

Fix memory corruption while configuring CPU windows on Marvell SoCsClosedPublicActions