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

Commit 7ea72dcd authored by Marco Nelissen's avatar Marco Nelissen
Browse files

Don't process objects with empty name 

as this will result in a bad database entry.

Bug: 31526567
Change-Id: I2e2a88ca98b189300fa2885b082746252f920bb4
parent 7b973eb8

media/mtp/MtpServer.cpp

+4 −0
Original line number Diff line number Diff line
@@ -927,6 +927,10 @@ MtpResponseCode MtpServer::doSendObjectInfo() { 
    if (!mData.getUInt32(temp32)) return MTP_RESPONSE_INVALID_PARAMETER;  // sequence number

    MtpStringBuffer name, created, modified;

    if (!mData.getString(name)) return MTP_RESPONSE_INVALID_PARAMETER;    // file name

    if (name.getCharCount() == 0) {

        ALOGE("empty name");

        return MTP_RESPONSE_INVALID_PARAMETER;

    }

    if (!mData.getString(created)) return MTP_RESPONSE_INVALID_PARAMETER;      // date created

    if (!mData.getString(modified)) return MTP_RESPONSE_INVALID_PARAMETER;     // date modified

    // keywords follow