12345678910111213141516171819202122232425262728293031323334353637383940 |
- From b960c0b43a080287a7c13533eeb2d9f288db1414 Mon Sep 17 00:00:00 2001
- From: Florian Festi <ffesti@redhat.com>
- Date: Thu, 16 Mar 2023 19:05:04 +0100
- Subject: [PATCH] Fix compiler error on clang
- Turns out variable declarations are not allowed after a label, even in
- C99. And while some compilers don't seem to care others do.
- Moving the declaration of mayopen to the start of the function to avoid
- this problem.
- Resolves: #2435
- Upstream: b960c0b43a080287a7c13533eeb2d9f288db1414
- Signed-off-by: James Knight <james.d.knight@live.com>
- ---
- lib/fsm.c | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
- diff --git a/lib/fsm.c b/lib/fsm.c
- index 5671ac642d..183293edb0 100644
- --- a/lib/fsm.c
- +++ b/lib/fsm.c
- @@ -879,6 +879,7 @@ int rpmPackageFilesInstall(rpmts ts, rpmte te, rpmfiles files,
- int nodigest = (rpmtsFlags(ts) & RPMTRANS_FLAG_NOFILEDIGEST) ? 1 : 0;
- int nofcaps = (rpmtsFlags(ts) & RPMTRANS_FLAG_NOCAPS) ? 1 : 0;
- int firstlinkfile = -1;
- + int mayopen = 0;
- char *tid = NULL;
- struct filedata_s *fdata = xcalloc(fc, sizeof(*fdata));
- struct filedata_s *firstlink = NULL;
- @@ -1016,7 +1017,7 @@ int rpmPackageFilesInstall(rpmts ts, rpmte te, rpmfiles files,
-
- setmeta:
- /* Special files require path-based ops */
- - int mayopen = S_ISREG(fp->sb.st_mode) || S_ISDIR(fp->sb.st_mode);
- + mayopen = S_ISREG(fp->sb.st_mode) || S_ISDIR(fp->sb.st_mode);
- if (!rc && fd == -1 && mayopen) {
- int flags = O_RDONLY;
- /* Only follow safe symlinks, and never on temporary files */
|