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

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

Check mprotect result 

am: 866c800c

Change-Id: I7b20d72aafb9a95480c1a6b16f0cafc4d1749470
parents f59ab362 866c800c

media/libmedia/IOMX.cpp

+28 −24
Original line number Diff line number Diff line
@@ -637,7 +637,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);
@@ -665,6 +668,7 @@ status_t BnOMX::onTransact( 
                                }

                            }

                        }

                    }

                } else {

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

                }