Remove unnecessary codes for checking size

diff --git a/src/NanHalEventCallbackHandler.cpp b/src/NanHalEventCallbackHandler.cpp
index 4762334f9490d319cd7fe48aead5991a3a45b5a9..7f36464a7d844fbe74ae7869ecf30bd7db32a315 100644 (file)
--- a/src/NanHalEventCallbackHandler.cpp
+++ b/src/NanHalEventCallbackHandler.cpp
@@ -240,8 +240,7 @@ void raiseServiceDiscoveredEvent(NanMatchInd *event)
        discoveryEvent.subId = event->publish_subscribe_id;
        discoveryEvent.requestorId = event->requestor_instance_id;
 
-       if (event->service_specific_info_len < 0
-                       || event->service_specific_info_len > NAN_MAX_SERVICE_SPECIFIC_INFO_LEN) {
+       if (event->service_specific_info_len > NAN_MAX_SERVICE_SPECIFIC_INFO_LEN) {
                NAN_LOGE("Invalid service_specific_info length");
                return;
        }
@@ -250,8 +249,7 @@ void raiseServiceDiscoveredEvent(NanMatchInd *event)
                        event->service_specific_info_len);
        discoveryEvent.specificInfoLen = event->service_specific_info_len;
 
-       if (event->sdf_match_filter_len < 0
-                       || event->sdf_match_filter_len > NAN_MAX_MATCH_FILTER_LEN) {
+       if (event->sdf_match_filter_len > NAN_MAX_MATCH_FILTER_LEN) {
                NAN_LOGE("Invalid sdf_match_filter length");
                return;
        }
@@ -299,8 +297,7 @@ void raiseFollowupEvent(NanFollowupInd *event)
        followupEvent.pubSubId = event->publish_subscribe_id;
        followupEvent.requestorId = event->requestor_instance_id;
 
-       if (event->service_specific_info_len < 0
-                       || event->service_specific_info_len > NAN_MAX_SERVICE_SPECIFIC_INFO_LEN) {
+       if (event->service_specific_info_len > NAN_MAX_SERVICE_SPECIFIC_INFO_LEN) {
                NAN_LOGE("Invalid service_specific_info length");
                return;
        }
@@ -349,8 +346,7 @@ void raiseDataPathRequestedEvent(NanDataPathRequestInd *event)
        dataPathEvent.serviceId = event->service_instance_id;
        dataPathEvent.dataPathId = event->ndp_instance_id;
 
-       if (event->app_info.ndp_app_info_len < 0
-                       || event->app_info.ndp_app_info_len > NAN_DP_MAX_APP_INFO_LEN) {
+       if (event->app_info.ndp_app_info_len > NAN_DP_MAX_APP_INFO_LEN) {
                NAN_LOGE("Invalid app_info length");
                return;
        }
@@ -386,8 +382,7 @@ void raiseDataPathConfirmedEvent(NanDataPathConfirmInd *event)
        NanDataPathConfirmEvent dataPathConfirmEvent;
        dataPathConfirmEvent.dataPathId = event->ndp_instance_id;
 
-       if (event->app_info.ndp_app_info_len < 0
-                       || event->app_info.ndp_app_info_len > NAN_DP_MAX_APP_INFO_LEN) {
+       if (event->app_info.ndp_app_info_len > NAN_DP_MAX_APP_INFO_LEN) {
                NAN_LOGE("Invalid app_info length");
                return;
        }