From: Jakub Skowron <j.skowron@samsung.com>
Date: Wed, 13 Jul 2016 08:58:17 +0000 (+0200)
Subject: [NFC] Prevent crash when nfc_ndef_record_get_uri fails to set out parameter
X-Git-Tag: submit/tizen/20160808.053811~13^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F67%2F79867%2F1;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[NFC] Prevent crash when nfc_ndef_record_get_uri fails to set out parameter

Currently nfc_ndef_record_get_uri sometimes returns NFC_ERROR_NONE
while not setting uri parameter correctly.

[Verification]
  Code compiles without errors.

Change-Id: I9e420f8317c561849bd241f36d767c9c17df85ad
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>
---

diff --git a/src/nfc/nfc_message_utils.cc b/src/nfc/nfc_message_utils.cc
index 5494b22..c5fc48a 100644
--- a/src/nfc/nfc_message_utils.cc
+++ b/src/nfc/nfc_message_utils.cc
@@ -767,7 +767,7 @@ static PlatformResult getURIFromHandle(nfc_ndef_record_h handle,
   LoggerD("Entered");
   char* uri = NULL;
   int result = nfc_ndef_record_get_uri(handle, &uri);
-  if (NFC_ERROR_NONE != result) {
+  if (NFC_ERROR_NONE != result || !uri) {
     LoggerE("Can't get record's uri: %d, %s", result,
             NFCUtil::getNFCErrorMessage(result).c_str());
     NFCMessageUtils::RemoveMessageHandle(message_handle);