Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0b001b52 authored by Marco Nelissen's avatar Marco Nelissen Committed by android-build-merger
Browse files

Check mprotect result am: 866c800c am: a0cfcf38 am: 638aca47 am:... 

Check mprotect result am: 866c800c am: a0cfcf38 am: 638aca47 am: e353ce4b am: 762b7a9b am: db8cfb97 am: dffa39ac am: 53b2b9eb am: 4073c59a
am: 093d1641

Change-Id: I9457dccead8d6315483a12bcaa620d21f39135ed
parents 0dcde535 093d1641

media/libmedia/IOMX.cpp

+28 −24
Original line number Diff line number Diff line
@@ -733,7 +733,10 @@ status_t BnOMX::onTransact( 
                            // mark the last page as inaccessible, to avoid exploitation

                            // of codecs that access past the end of the allocation because

                            // they didn't check the size

                            mprotect((char*)params + allocSize - pageSize, pageSize, PROT_NONE);

                            if (mprotect((char*)params + allocSize - pageSize, pageSize,

                                    PROT_NONE) != 0) {

                                ALOGE("mprotect failed: %s", strerror(errno));

                            } else {

                                switch (code) {

                                    case GET_PARAMETER:

                                        err = getParameter(node, index, params, size);
@@ -761,6 +764,7 @@ status_t BnOMX::onTransact( 
                                }

                            }

                        }

                    }

                } else {

                    ALOGE("couldn't map: %s", strerror(errno));

                }