From 5e083dcc202e53f21a9b2ca786d939abed090bde Mon Sep 17 00:00:00 2001 From: Ethan Yonker Date: Wed, 3 Sep 2014 14:46:41 -0500 Subject: Improve MtpStorage handling when more than one storage is present Change-Id: I02f1984d02a85d9fd42450cc3772e88a43d9e9ff --- mtp/mtp_MtpDatabase.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'mtp/mtp_MtpDatabase.cpp') diff --git a/mtp/mtp_MtpDatabase.cpp b/mtp/mtp_MtpDatabase.cpp index 60a871ce3..acdab523d 100755 --- a/mtp/mtp_MtpDatabase.cpp +++ b/mtp/mtp_MtpDatabase.cpp @@ -322,7 +322,7 @@ MtpResponseCode MyMtpDatabase::getObjectPropertyValue(MtpObjectHandle handle, MtpDataPacket& packet) { MTPD("MyMtpDatabase::getObjectPropertyValue mtpid: %i, property: %x\n", handle, property); int type; - MtpResponseCode result; + MtpResponseCode result = MTP_RESPONSE_INVALID_OBJECT_HANDLE; uint64_t longValue; if (!getObjectPropertyInfo(property, type)) { MTPE("MyMtpDatabase::setObjectPropertyValue returning MTP_RESPONSE_OBJECT_PROP_NOT_SUPPORTED\n"); @@ -336,6 +336,11 @@ MtpResponseCode MyMtpDatabase::getObjectPropertyValue(MtpObjectHandle handle, } } + if (result != MTP_RESPONSE_OK) { + MTPE("MyMtpDatabase::setObjectPropertyValue unable to locate handle: %i\n", handle); + return MTP_RESPONSE_INVALID_OBJECT_HANDLE; + } + // special case date properties, which are strings to MTP // but stored internally as a uint64 if (property == MTP_PROPERTY_DATE_MODIFIED || property == MTP_PROPERTY_DATE_ADDED) { -- cgit v1.2.3