diff options
| author | Vignesh Raman <Vignesh_Raman@mentor.com> | 2016-10-07 12:47:12 +0530 |
|---|---|---|
| committer | Cosmin Cernat <cosmin.cernat@continental-corporation.com> | 2017-01-26 16:36:27 +0200 |
| commit | fca84b3b67600e58d75f738cbe2cf0186b674777 (patch) | |
| tree | 1b706d44222374843b2ba6d8625fbb26f5aba0e5 | |
| parent | 624d40261a94313356259155959ea07ecaf51d5a (diff) | |
| download | persistence-administrator-fca84b3b67600e58d75f738cbe2cf0186b674777.tar.gz | |
PAS: Limitation of user_no to 4 is removed
Change-Id: I3d39d9dee86a0aa055f04f81ea825018014651ed
Signed-off-by: Ingo Huerner <Ingo_Huerner@mentor.com>
| -rw-r--r-- | Administrator/src/ssw_pers_admin_access_lib.c | 6 | ||||
| -rw-r--r-- | Administrator/src/ssw_pers_admin_backup.c | 4 |
2 files changed, 6 insertions, 4 deletions
diff --git a/Administrator/src/ssw_pers_admin_access_lib.c b/Administrator/src/ssw_pers_admin_access_lib.c index 4abaf37..654ab31 100644 --- a/Administrator/src/ssw_pers_admin_access_lib.c +++ b/Administrator/src/ssw_pers_admin_access_lib.c @@ -32,6 +32,8 @@ #include <sys/stat.h> #include <mqueue.h> #include <semaphore.h> +#include <limits.h> + #include "persistence_admin_service.h" #include "ssw_pers_admin_access_lib.h" @@ -946,7 +948,7 @@ long persAdminUserDataCopy(unsigned int src_user_no, unsigned int src_seat_no, u long errorCode = PAS_FAILURE ; /* check params */ - if((src_user_no <= 4) && (src_seat_no <= 4) && (dest_user_no <= 4) && (dest_seat_no <= 4)) + if((src_user_no <= UINT_MAX) && (src_seat_no <= 4) && (dest_user_no <= UINT_MAX) && (dest_seat_no <= 4)) { } @@ -1059,7 +1061,7 @@ long persAdminUserDataDelete(unsigned int user_no, unsigned int seat_no) long errorCode = PAS_FAILURE ; /* check params */ - if((user_no <= 4) && (seat_no <= 4)) + if((user_no <= UINT_MAX) && (seat_no <= 4)) { } diff --git a/Administrator/src/ssw_pers_admin_backup.c b/Administrator/src/ssw_pers_admin_backup.c index d6fb924..aa14c51 100644 --- a/Administrator/src/ssw_pers_admin_backup.c +++ b/Administrator/src/ssw_pers_admin_backup.c @@ -33,7 +33,7 @@ #include "ssw_pers_admin_service.h" #include "persComDataOrg.h" #include "persComTypes.h" - +#include <limits.h> /* L&T context */ @@ -53,7 +53,7 @@ DLT_IMPORT_CONTEXT (persAdminSvcDLTCtx) #define NAMES_APP_MAX_SIZE (NAME_APP_MAX_SIZE * 100) #define IS_VALID_APPL(applicationID) ((NIL != (applicationID)) && (0 != (*(applicationID)))) -#define IS_VALID_USER(user_no) ((PERSIST_SELECT_ALL_USERS == (user_no)) || ((1 <= (user_no)) && (4 >= (user_no)))) +#define IS_VALID_USER(user_no) ((PERSIST_SELECT_ALL_USERS == (user_no)) || ((1 <= (user_no)) && (UINT_MAX >= (user_no)))) #define IS_VALID_SEAT(seat_no) ((PERSIST_SELECT_ALL_SEATS == (seat_no)) || ((1 <= (seat_no)) && (4 >= (seat_no)))) const char* gSharedPublicRelativePath = PERS_ORG_SHARED_FOLDER_NAME PERS_ORG_PUBLIC_FOLDER_NAME_; |