diff options
| author | Amber Bhardwaj <amber.bhardwaj10@gmail.com> | 2017-06-16 11:08:42 +0530 |
|---|---|---|
| committer | Christoph Lipka <clipka@users.noreply.github.com> | 2017-06-16 07:38:42 +0200 |
| commit | cf65615d35b7c0d3cae3239c53374d841d9a175d (patch) | |
| tree | 16693b958e879a36c737cb50f13fa5b3ae8e7ce2 /src/kpi/dlt-kpi-common.c | |
| parent | b91b38534e5276db69ffbf938177c227b01c65ad (diff) | |
| download | DLT-daemon-cf65615d35b7c0d3cae3239c53374d841d9a175d.tar.gz | |
Input parameter check & Error message modification (#15)
* Update dlt-kpi-process.c
Added input param checking and added return value in non-void functions.
* Update dlt-kpi.c
Added Input param checking and added return values in non-void functions.
* Input parameter check and Error msg modification
Added null pointer checking
Added input parameter checking
Added return values in non-void functions
Added proper error message for debugging purpose
Signed-off-by: amberbhardwaj <amber.bhardwaj10@gmail.com>
Diffstat (limited to 'src/kpi/dlt-kpi-common.c')
| -rw-r--r-- | src/kpi/dlt-kpi-common.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/kpi/dlt-kpi-common.c b/src/kpi/dlt-kpi-common.c index 26c4e28..fa131dc 100644 --- a/src/kpi/dlt-kpi-common.c +++ b/src/kpi/dlt-kpi-common.c @@ -30,7 +30,13 @@ static int dlt_kpi_cpu_count = -1; DltReturnValue dlt_kpi_read_file_compact(char *filename, char **target) { - char buffer[BUFFER_SIZE]; + if(filename == NULL || target == NULL) + { + fprintf(stderr, "%s: Nullpointer parameter!\n",__func__); + return DLT_RETURN_WRONG_PARAMETER; + } + + char buffer[BUFFER_SIZE]; int ret = dlt_kpi_read_file(filename, buffer, BUFFER_SIZE); if(ret < DLT_RETURN_OK) return ret; @@ -50,7 +56,7 @@ DltReturnValue dlt_kpi_read_file(char* filename, char* buffer, uint maxLength) { if(filename == NULL || buffer == NULL) { - fprintf(stderr, "Nullpointer parameter!\n"); + fprintf(stderr, "%s: Nullpointer parameter!\n",__func__); return DLT_RETURN_WRONG_PARAMETER; } |