From 602896277c76298bcbc152312d915dec306eda27 Mon Sep 17 00:00:00 2001 From: Ton Voon Date: Mon, 21 Jun 2010 13:03:24 +0100 Subject: Change np_state_write_string() to not be a pointer diff --git a/lib/tests/test_utils.c b/lib/tests/test_utils.c index 6e04bfc..e90d4fb 100644 --- a/lib/tests/test_utils.c +++ b/lib/tests/test_utils.c @@ -355,7 +355,7 @@ main (int argc, char **argv) unlink("var/generated"); temp_state_key->_filename="var/generated"; current_time=1234567890; - np_state_write_string(¤t_time, "String to read"); + np_state_write_string(current_time, "String to read"); ok(system("cmp var/generated var/statefile")==0, "Generated file same as expected"); @@ -365,20 +365,20 @@ main (int argc, char **argv) unlink("var/generated_directory"); temp_state_key->_filename="var/generated_directory/statefile"; current_time=1234567890; - np_state_write_string(¤t_time, "String to read"); + np_state_write_string(current_time, "String to read"); ok(system("cmp var/generated_directory/statefile var/statefile")==0, "Have created directory"); /* This test to check cannot write to dir - can't automate yet */ /* unlink("var/generated_bad_dir"); mkdir("var/generated_bad_dir", S_IRUSR); - np_state_write_string(¤t_time, "String to read"); + np_state_write_string(current_time, "String to read"); */ temp_state_key->_filename="var/generated"; time(¤t_time); - np_state_write_string(NULL, "String to read"); + np_state_write_string(0, "String to read"); temp_state_data = np_state_read(temp_state_key); /* Check time is set to current_time */ ok(system("cmp var/generated var/statefile > /dev/null")!=0, "Generated file should be different this time"); @@ -388,7 +388,7 @@ main (int argc, char **argv) /* Don't know how to automatically test this. Need to be able to redefine die and catch the error */ /* temp_state_key->_filename="/dev/do/not/expect/to/be/able/to/write"; - np_state_write_string(NULL, "Bad file"); + np_state_write_string(0, "Bad file"); */ diff --git a/lib/utils_base.c b/lib/utils_base.c index 5eaf57b..e6b20c8 100644 --- a/lib/utils_base.c +++ b/lib/utils_base.c @@ -519,7 +519,7 @@ int _np_state_read_file(FILE *f) { * two things writing to same key at same time. * Will die with UNKNOWN if errors */ -void np_state_write_string(time_t *data_time, char *data_string) { +void np_state_write_string(time_t data_time, char *data_string) { FILE *fp; char *temp_file=NULL; int fd=0, result=0; @@ -528,10 +528,10 @@ void np_state_write_string(time_t *data_time, char *data_string) { char *directories=NULL; char *p=NULL; - if(data_time==NULL) + if(data_time==0) time(¤t_time); else - current_time=*data_time; + current_time=data_time; /* If file doesn't currently exist, create directories */ if(access(this_nagios_plugin->state->_filename,F_OK)!=0) { diff --git a/lib/utils_base.h b/lib/utils_base.h index 12576d7..8f2f664 100644 --- a/lib/utils_base.h +++ b/lib/utils_base.h @@ -95,7 +95,7 @@ char *np_extract_value(const char*, const char*, char); void np_enable_state(char *, int); state_data *np_state_read(); -void np_state_write_string(time_t *, char *); +void np_state_write_string(time_t, char *); void np_init(char *); void np_cleanup(); -- cgit v0.10-9-g596f