Skip to content
Snippets Groups Projects
Commit 9d060033 authored by Morris Jette's avatar Morris Jette
Browse files

Initialize some variabled before use

Make sure variables are initialized before passing a pointer to
  them as a function argument. The variable is set in the underlying
  function, so this is mostly cosmetic.
parent ad66d638
No related branches found
No related tags found
No related merge requests found
......@@ -246,7 +246,7 @@ extern char *get_licenses_used(void)
/* Initialize licenses on this system based upon slurm.conf */
extern int license_init(char *licenses)
{
bool valid;
bool valid = true;
last_license_update = time(NULL);
......@@ -270,7 +270,7 @@ extern int license_update(char *licenses)
ListIterator iter;
licenses_t *license_entry, *match;
List new_list;
bool valid;
bool valid = true;
new_list = _build_license_list(licenses, &valid);
if (!valid)
......@@ -303,11 +303,11 @@ extern int license_update(char *licenses)
if (!match) {
info("license %s removed with %u in use",
license_entry->name, license_entry->used);
license_entry->name, license_entry->used);
} else {
if (license_entry->used > match->total) {
info("license %s count decreased",
match->name);
match->name);
}
}
}
......@@ -567,7 +567,7 @@ extern List license_validate(char *licenses, bool *valid)
*/
extern void license_job_merge(struct job_record *job_ptr)
{
bool valid;
bool valid = true;
FREE_NULL_LIST(job_ptr->license_list);
job_ptr->license_list = _build_license_list(job_ptr->licenses, &valid);
......
......@@ -1929,7 +1929,7 @@ static int _restore_job_dependencies(void)
struct job_record *job_ptr;
ListIterator job_iterator;
char *new_depend;
bool valid;
bool valid = true;
List license_list;
assoc_mgr_clear_used_info();
......
......@@ -1734,7 +1734,7 @@ extern int create_resv(resv_desc_msg_t *resv_desc_ptr)
goto bad_parse;
}
if (resv_desc_ptr->licenses) {
bool valid;
bool valid = true;
license_list = _license_validate2(resv_desc_ptr, &valid);
if (!valid) {
info("Reservation request has invalid licenses %s",
......@@ -2704,7 +2704,7 @@ static bool _validate_one_reservation(slurmctld_resv_t *resv_ptr)
resv_ptr->account_not = account_not;
}
if (resv_ptr->licenses) {
bool valid;
bool valid = true;
if (resv_ptr->license_list)
list_destroy(resv_ptr->license_list);
resv_ptr->license_list = license_validate(resv_ptr->licenses,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment