From 77fcdd8e00e6934d4e503aaf9743d563f249129d Mon Sep 17 00:00:00 2001
From: Mamoru TASAKA <mtasaka@fedoraproject.org>
Date: Thu, 7 Jan 2021 20:53:55 +0900
Subject: [PATCH] Fix build failure with ruby 3.0
With ruby 3.0, build fails like:
```
/builddir/build/BUILD/dislocker-0.7.3/src/config.c: In function 'setclearkey':
/builddir/build/BUILD/dislocker-0.7.3/src/config.c:59:13: error: expected identifier or '(' before numeric constant
59 | int true = TRUE;
```
This is because ruby 3.0 header will include <stdbool.h>, which defines
"true"/"false", ref:
https://pubs.opengroup.org/onlinepubs/009695399/basedefs/stdbool.h.html
So using "true" as variable must be renamed.
---
src/config.c | 50 +++++++++++++++++++++++++-------------------------
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/src/config.c b/src/config.c
index 2f1bdbc..f9b9423 100644
--- a/src/config.c
+++ b/src/config.c
@@ -56,13 +56,13 @@ static void hide_opt(char* opt)
static void setclearkey(dis_context_t dis_ctx, char* optarg)
{
(void) optarg;
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_USE_CLEAR_KEY, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_USE_CLEAR_KEY, &trueval);
}
static void setbekfile(dis_context_t dis_ctx, char* optarg)
{
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_USE_BEK_FILE, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_USE_BEK_FILE, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_BEK_FILE_PATH, optarg);
}
static void setforceblock(dis_context_t dis_ctx, char* optarg)
@@ -76,14 +76,14 @@ static void setforceblock(dis_context_t dis_ctx, char* optarg)
}
static void setfvek(dis_context_t dis_ctx, char* optarg)
{
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_USE_FVEK_FILE, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_USE_FVEK_FILE, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_FVEK_FILE_PATH, optarg);
}
static void setvmk(dis_context_t dis_ctx, char* optarg)
{
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_USE_VMK_FILE, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_USE_VMK_FILE, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_VMK_FILE_PATH, optarg);
}
static void setlogfile(dis_context_t dis_ctx, char* optarg)
@@ -97,8 +97,8 @@ static void setoffset(dis_context_t dis_ctx, char* optarg)
}
static void setrecoverypwd(dis_context_t dis_ctx, char* optarg)
{
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_USE_RECOVERY_PASSWORD, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_USE_RECOVERY_PASSWORD, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_RECOVERY_PASSWORD, optarg);
hide_opt(optarg);
}
@@ -111,19 +111,19 @@ static void setquiet(dis_context_t dis_ctx, char* optarg)
static void setro(dis_context_t dis_ctx, char* optarg)
{
(void) optarg;
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_READ_ONLY, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_READ_ONLY, &trueval);
}
static void setstateok(dis_context_t dis_ctx, char* optarg)
{
(void) optarg;
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_DONT_CHECK_VOLUME_STATE, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_DONT_CHECK_VOLUME_STATE, &trueval);
}
static void setuserpassword(dis_context_t dis_ctx, char* optarg)
{
- int true = TRUE;
- dis_setopt(dis_ctx, DIS_OPT_USE_USER_PASSWORD, &true);
+ int trueval = TRUE;
+ dis_setopt(dis_ctx, DIS_OPT_USE_USER_PASSWORD, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_USER_PASSWORD, optarg);
hide_opt(optarg);
}
@@ -266,7 +266,7 @@ int dis_getopts(dis_context_t dis_ctx, int argc, char** argv)
return -1;
dis_config_t* cfg = &dis_ctx->cfg;
- int true = TRUE;
+ int trueval = TRUE;
long_opts = malloc(nb_options * sizeof(struct option));
@@ -285,12 +285,12 @@ int dis_getopts(dis_context_t dis_ctx, int argc, char** argv)
{
case 'c':
{
- dis_setopt(dis_ctx, DIS_OPT_USE_CLEAR_KEY, &true);
+ dis_setopt(dis_ctx, DIS_OPT_USE_CLEAR_KEY, &trueval);
break;
}
case 'f':
{
- dis_setopt(dis_ctx, DIS_OPT_USE_BEK_FILE, &true);
+ dis_setopt(dis_ctx, DIS_OPT_USE_BEK_FILE, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_BEK_FILE_PATH, optarg);
break;
}
@@ -312,13 +312,13 @@ int dis_getopts(dis_context_t dis_ctx, int argc, char** argv)
}
case 'k':
{
- dis_setopt(dis_ctx, DIS_OPT_USE_FVEK_FILE, &true);
+ dis_setopt(dis_ctx, DIS_OPT_USE_FVEK_FILE, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_FVEK_FILE_PATH, optarg);
break;
}
case 'K':
{
- dis_setopt(dis_ctx, DIS_OPT_USE_VMK_FILE, &true);
+ dis_setopt(dis_ctx, DIS_OPT_USE_VMK_FILE, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_VMK_FILE_PATH, optarg);
break;
}
@@ -340,7 +340,7 @@ int dis_getopts(dis_context_t dis_ctx, int argc, char** argv)
}
case 'p':
{
- dis_setopt(dis_ctx, DIS_OPT_USE_RECOVERY_PASSWORD, &true);
+ dis_setopt(dis_ctx, DIS_OPT_USE_RECOVERY_PASSWORD, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_RECOVERY_PASSWORD, optarg);
hide_opt(optarg);
break;
@@ -353,17 +353,17 @@ int dis_getopts(dis_context_t dis_ctx, int argc, char** argv)
}
case 'r':
{
- dis_setopt(dis_ctx, DIS_OPT_READ_ONLY, &true);
+ dis_setopt(dis_ctx, DIS_OPT_READ_ONLY, &trueval);
break;
}
case 's':
{
- dis_setopt(dis_ctx, DIS_OPT_DONT_CHECK_VOLUME_STATE, &true);
+ dis_setopt(dis_ctx, DIS_OPT_DONT_CHECK_VOLUME_STATE, &trueval);
break;
}
case 'u':
{
- dis_setopt(dis_ctx, DIS_OPT_USE_USER_PASSWORD, &true);
+ dis_setopt(dis_ctx, DIS_OPT_USE_USER_PASSWORD, &trueval);
dis_setopt(dis_ctx, DIS_OPT_SET_USER_PASSWORD, optarg);
hide_opt(optarg);
break;
|
