Fix missing null checks 12/238112/4
authorMateusz Cegielka <m.cegielka@samsung.com>
Wed, 8 Jul 2020 12:55:27 +0000 (14:55 +0200)
committerMateusz Cegielka <m.cegielka@samsung.com>
Mon, 20 Jul 2020 11:42:47 +0000 (13:42 +0200)
Change-Id: I2fcc2622e9f39e71a6a2078e1248c69d7754ad5e

src/client/api/askuser-notification-client.cpp

index a65c08103d1e9c4baf8aa6d9360daaeb4335bf12..57686ed8a855d40c6d99b9019458efba8f49304a 100644 (file)
@@ -51,7 +51,7 @@ API
 int askuser_client_initialize(askuser_client **pp_client,
                               askuser_status_callback status_callback, void *p_user_data)
 {
-    if (!pp_client) {
+    if (!pp_client || !status_callback) {
         return ASKUSER_API_INVALID_PARAM;
     }
 
@@ -120,7 +120,7 @@ int askuser_client_check_app_privilege(askuser_client *p_client,
                                        const char *privilege,
                                        askuser_check_result *p_result)
 {
-    if (!p_client || !privilege || !p_result) {
+    if (!p_client || !app_id || !privilege || !p_result) {
         return ASKUSER_API_INVALID_PARAM;
     }
 
@@ -157,7 +157,7 @@ int askuser_client_check_app_privileges(askuser_client *p_client, const char *ap
                                         const char **privileges, size_t privileges_count,
                                         askuser_check_result *p_results)
 {
-    if (!p_client || !privileges || !p_results ||
+    if (!p_client || !app_id || !privileges || !p_results ||
         privileges_count == 0 || privileges_count > AskUser::Protocol::MAX_PRIVS_NUMBER) {
         return ASKUSER_API_INVALID_PARAM;
     }
@@ -243,7 +243,8 @@ int askuser_popup_send_ext_response(askuser_client *p_client, int popup_id,
                             const askuser_popup_result responses[],
                             size_t privacies_count)
 {
-    if (((!privacies || !responses) && privacies_count > 0) || (privacies_count == 0 && (privacies || responses))) {
+    if (!p_client || ((!privacies || !responses) && privacies_count > 0) ||
+        (privacies_count == 0 && (privacies || responses))) {
         return ASKUSER_API_INVALID_PARAM;
     }