diff options
| author | qiyao <qiyao> | 2012-07-20 08:59:31 +0000 |
|---|---|---|
| committer | qiyao <qiyao> | 2012-07-20 08:59:31 +0000 |
| commit | 507c8f61614dbdd7d859178d57b59d36e37fce16 (patch) | |
| tree | 66397635a95a696bda3c5887c685e7dd86a26e7d /gdb/cli/cli-setshow.c | |
| parent | 120b3ce0dbbb0fd1ce23237927fd481233490f2e (diff) | |
| download | gdb-507c8f61614dbdd7d859178d57b59d36e37fce16.tar.gz | |
gdb/
* cli/cli-setshow.c (do_setshow_command): Don't check pointer is
NULL for xfree.
Diffstat (limited to 'gdb/cli/cli-setshow.c')
| -rw-r--r-- | gdb/cli/cli-setshow.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/gdb/cli/cli-setshow.c b/gdb/cli/cli-setshow.c index dccf4255429..5ec7b55881f 100644 --- a/gdb/cli/cli-setshow.c +++ b/gdb/cli/cli-setshow.c @@ -170,16 +170,14 @@ do_setshow_command (char *arg, int from_tty, struct cmd_list_element *c) #endif *q++ = '\0'; new = (char *) xrealloc (new, q - new); - if (*(char **) c->var != NULL) - xfree (*(char **) c->var); + xfree (*(char **) c->var); *(char **) c->var = new; } break; case var_string_noescape: if (arg == NULL) arg = ""; - if (*(char **) c->var != NULL) - xfree (*(char **) c->var); + xfree (*(char **) c->var); *(char **) c->var = xstrdup (arg); break; case var_filename: @@ -187,8 +185,7 @@ do_setshow_command (char *arg, int from_tty, struct cmd_list_element *c) error_no_arg (_("filename to set it to.")); /* FALLTHROUGH */ case var_optional_filename: - if (*(char **) c->var != NULL) - xfree (*(char **) c->var); + xfree (*(char **) c->var); if (arg != NULL) { |