dcavalca / rpms / rpm

Forked from rpms/rpm 2 years ago
Clone
Blob Blame History Raw
From f525681b4f66026578bc728b864bfea3d814c29e Mon Sep 17 00:00:00 2001
From: Jes Sorensen <jsorensen@fb.com>
Date: Fri, 27 Mar 2020 18:31:36 -0400
Subject: [PATCH 16/33] Add basic autoconf and framework for fsverity support

Use the same signing key argument as is used for IMA file signing.

Signed-off-by: Jes Sorensen <jsorensen@fb.com>
---
 configure.ac     | 19 +++++++++++++++++++
 rpmsign.c        | 20 ++++++++++++++------
 sign/Makefile.am |  5 +++++
 sign/rpmsign.h   |  1 +
 4 files changed, 39 insertions(+), 6 deletions(-)

diff --git a/configure.ac b/configure.ac
index 3c102d5eb..cc7144440 100644
--- a/configure.ac
+++ b/configure.ac
@@ -919,6 +919,25 @@ fi
 AM_CONDITIONAL(WITH_IMAEVM,[test "$with_imaevm" = yes])
 AC_SUBST(WITH_IMAEVM_LIB)
 
+# fsverity
+AC_ARG_WITH([fsverity], [AS_HELP_STRING([--with-fsverity],[build with fsverity support])],[],[with_fsverity=no])
+if test "$with_fsverity" = yes ; then
+  AC_MSG_CHECKING([libfsverity])
+  AC_COMPILE_IFELSE(
+    [AC_LANG_PROGRAM(
+      [[#include <libfsverity.h>]],
+      [[return libfsverity_sign_digest(NULL, NULL, NULL, NULL);]]
+    )],
+    [AC_MSG_RESULT(yes)
+      AC_DEFINE(WITH_FSVERITY, 1, [Build with fsverity support?])
+      WITH_FSVERITY_LIB="-lfsverity"
+    ],
+    [AC_MSG_ERROR([--with-fsverity given, libfsverity or libfsverity.h missing])]
+  )
+fi
+AM_CONDITIONAL(WITH_FSVERITY,[test "$with_fsverity" = yes])
+AC_SUBST(WITH_FSVERITY_LIB)
+
 # libcap
 WITH_CAP_LIB=
 AC_ARG_WITH(cap, [AS_HELP_STRING([--with-cap],[build with capability support])],
diff --git a/rpmsign.c b/rpmsign.c
index e1d207da5..8861c2c59 100644
--- a/rpmsign.c
+++ b/rpmsign.c
@@ -18,7 +18,7 @@ enum modes {
 
 static int mode = MODE_NONE;
 
-#ifdef WITH_IMAEVM
+#if defined(WITH_IMAEVM) || defined(WITH_FSVERITY)
 static int fskpass = 0;
 static char * fileSigningKey = NULL;
 #endif
@@ -39,6 +39,13 @@ static struct poptOption signOptsTable[] = {
     { "signfiles", '\0', (POPT_ARG_VAL|POPT_ARGFLAG_OR),
 	&sargs.signflags, RPMSIGN_FLAG_IMA,
 	N_("sign package(s) files"), NULL},
+#endif
+#ifdef WITH_FSVERITY
+    { "signverity", '\0', (POPT_ARG_VAL|POPT_ARGFLAG_OR),
+	&sargs.signflags, RPMSIGN_FLAG_FSVERITY,
+	N_("generate fsverity signatures for package(s) files"), NULL},
+#endif
+#if defined(WITH_IMAEVM) || defined(WITH_FSVERITY)
     { "fskpath", '\0', POPT_ARG_STRING, &fileSigningKey, 0,
 	N_("use file signing key <key>"),
 	N_("<key>") },
@@ -59,7 +66,7 @@ static struct poptOption optionsTable[] = {
     POPT_TABLEEND
 };
 
-#ifdef WITH_IMAEVM
+#if defined(WITH_IMAEVM) || defined(WITH_FSVERITY)
 static char *get_fskpass(void)
 {
     struct termios flags, tmp_flags;
@@ -106,12 +113,12 @@ static int doSign(poptContext optCon, struct rpmSignArgs *sargs)
 	goto exit;
     }
 
-#ifdef WITH_IMAEVM
+#if defined(WITH_IMAEVM) || defined(WITH_FSVERITY)
     if (fileSigningKey) {
 	rpmPushMacro(NULL, "_file_signing_key", NULL, fileSigningKey, RMIL_GLOBAL);
     }
 
-    if (sargs->signflags & RPMSIGN_FLAG_IMA) {
+    if (sargs->signflags & (RPMSIGN_FLAG_IMA | RPMSIGN_FLAG_FSVERITY)) {
 	char *fileSigningKeyPassword = NULL;
 	char *key = rpmExpand("%{?_file_signing_key}", NULL);
 	if (rstreq(key, "")) {
@@ -165,8 +172,9 @@ int main(int argc, char *argv[])
 	argerror(_("no arguments given"));
     }
 
-#ifdef WITH_IMAEVM
-    if (fileSigningKey && !(sargs.signflags & RPMSIGN_FLAG_IMA)) {
+#if defined(WITH_IMAEVM) || defined(WITH_FSVERITY)
+    if (fileSigningKey &&
+	!(sargs.signflags & (RPMSIGN_FLAG_IMA | RPMSIGN_FLAG_FSVERITY))) {
 	argerror(_("--fskpath may only be specified when signing files"));
     }
 #endif
diff --git a/sign/Makefile.am b/sign/Makefile.am
index db774de0e..8d372915a 100644
--- a/sign/Makefile.am
+++ b/sign/Makefile.am
@@ -24,3 +24,8 @@ if WITH_IMAEVM
 librpmsign_la_SOURCES += rpmsignfiles.c rpmsignfiles.h
 librpmsign_la_LIBADD += @WITH_IMAEVM_LIB@
 endif
+
+if WITH_FSVERITY
+librpmsign_la_SOURCES += rpmsignverity.c rpmsignverity.h
+librpmsign_la_LIBADD += @WITH_FSVERITY_LIB@
+endif
diff --git a/sign/rpmsign.h b/sign/rpmsign.h
index 7a770d879..2b8a10a1a 100644
--- a/sign/rpmsign.h
+++ b/sign/rpmsign.h
@@ -17,6 +17,7 @@ enum rpmSignFlags_e {
     RPMSIGN_FLAG_NONE		= 0,
     RPMSIGN_FLAG_IMA		= (1 << 0),
     RPMSIGN_FLAG_RPMV3		= (1 << 1),
+    RPMSIGN_FLAG_FSVERITY	= (1 << 2),
 };
 typedef rpmFlags rpmSignFlags;
 
-- 
2.27.0