From 01ab0b79a1fa25557a625757ad37a0f5d74f28ad Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Fri, 6 Jan 2006 10:14:19 +0000 Subject: (rm_1): Remove `static' attribute on local `status'. First off, the attribute should have been `volatile' (not static) to avoid longjmp-related risk of clobber. Secondly, now there is no longer any risk of a local variable being clobbered, so there's no need for any attribute at all. --- src/remove.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) (limited to 'src') diff --git a/src/remove.c b/src/remove.c index 208f56997..d146ea203 100644 --- a/src/remove.c +++ b/src/remove.c @@ -1309,11 +1309,8 @@ rm_1 (Dirstack_state *ds, char const *filename, AD_push_initial (ds); AD_INIT_OTHER_MEMBERS (); - /* Put `status' in static storage, so it can't be clobbered - by the potential longjmp into this function. */ - static enum RM_status status; int fd_cwd = AT_FDCWD; - status = remove_entry (fd_cwd, ds, filename, x, NULL); + enum RM_status status = remove_entry (fd_cwd, ds, filename, x, NULL); if (status == RM_NONEMPTY_DIR) { /* In the event that remove_dir->remove_cwd_entries detects -- cgit v1.2.3-70-g09d2