package/boot/efivar/posix-compat.patch.musl

   1 # --- T2-COPYRIGHT-NOTE-BEGIN ---
   2 # T2 SDE: package/*/efivar/posix-compat.patch.musl
   3 # Copyright (C) 2023 The T2 SDE Project
   4 #
   5 # This Copyright note is generated by scripts/Create-CopyPatch,
   6 # more information can be found in the files COPYING and README.
   7 #
   8 # This patch file is dual-licensed. It is available under the license the
   9 # patched project is licensed under, as long as it is an OpenSource license
  10 # as defined at http://www.opensource.org/ (e.g. BSD, X11) or under the terms
  11 # of the GNU General Public License version 2 as used by the T2 SDE.
  12 # --- T2-COPYRIGHT-NOTE-END ---
  13
  14 From cece3ffd5be2f8641eb694513f2b73e5eb97ffd3 Mon Sep 17 00:00:00 2001
  15 From: Natanael Copa <ncopa@alpinelinux.org>
  16 Date: Fri, 28 Jan 2022 12:13:30 +0100
  17 Subject: [PATCH 1/2] efisecdb: fix build with musl libc
  18
  19 Refactor code to use POSIX atexit(3) instead of the GNU specific
  20 on_exit(3).
  21
  22 Resolves: #197
  23 Resolves: #202
  24 Signed-off-by: Natanael Copa <ncopa@alpinelinux.org>
  25 ---
  26  src/compiler.h |  2 --
  27  src/efisecdb.c | 68 +++++++++++++++++++-------------------------------
  28  2 files changed, 26 insertions(+), 44 deletions(-)
  29
  30 diff --git a/src/compiler.h b/src/compiler.h
  31 index e2f18f0b..d95fb014 100644
  32 --- a/src/compiler.h
  33 +++ b/src/compiler.h
  34 @@ -7,8 +7,6 @@
  35  #ifndef COMPILER_H_
  36  #define COMPILER_H_
  37
  38 -#include <sys/cdefs.h>
  39 -
  40  /* GCC version checking borrowed from glibc. */
  41  #if defined(__GNUC__) && defined(__GNUC_MINOR__)
  42  #  define GNUC_PREREQ(maj,min) \
  43 diff --git a/src/efisecdb.c b/src/efisecdb.c
  44 index f8823737..6bd5ad90 100644
  45 --- a/src/efisecdb.c
  46 +++ b/src/efisecdb.c
  47 @@ -25,6 +25,10 @@
  48  extern char *optarg;
  49  extern int optind, opterr, optopt;
  50
  51 +static efi_secdb_t *secdb = NULL;
  52 +static list_t infiles;
  53 +static list_t actions;
  54 +
  55  struct hash_param {
  56         char *name;
  57         efi_secdb_type_t algorithm;
  58 @@ -187,12 +191,11 @@ add_action(list_t *list, action_type_t action_type, const efi_guid_t *owner,
  59  }
  60
  61  static void
  62 -free_actions(int status UNUSED, void *actionsp)
  63 +free_actions(void)
  64  {
  65 -       list_t *actions = (list_t *)actionsp;
  66         list_t *pos, *tmp;
  67
  68 -       for_each_action_safe(pos, tmp, actions) {
  69 +       for_each_action_safe(pos, tmp, &actions) {
  70                 action_t *action = list_entry(pos, action_t, list);
  71
  72                 list_del(&action->list);
  73 @@ -202,12 +205,11 @@ free_actions(int status UNUSED, void *actionsp)
  74  }
  75
  76  static void
  77 -free_infiles(int status UNUSED, void *infilesp)
  78 +free_infiles(void)
  79  {
  80 -       list_t *infiles = (list_t *)infilesp;
  81         list_t *pos, *tmp;
  82
  83 -       for_each_ptr_safe(pos, tmp, infiles) {
  84 +       for_each_ptr_safe(pos, tmp, &infiles) {
  85                 ptrlist_t *entry = list_entry(pos, ptrlist_t, list);
  86
  87                 list_del(&entry->list);
  88 @@ -216,27 +218,12 @@ free_infiles(int status UNUSED, void *infilesp)
  89  }
  90
  91  static void
  92 -maybe_free_secdb(int status UNUSED, void *voidp)
  93 +maybe_free_secdb(void)
  94  {
  95 -       efi_secdb_t **secdbp = (efi_secdb_t **)voidp;
  96 -
  97 -       if (secdbp == NULL || *secdbp == NULL)
  98 +       if (secdb == NULL)
  99                 return;
 100
 101 -       efi_secdb_free(*secdbp);
 102 -}
 103 -
 104 -static void
 105 -maybe_do_unlink(int status, void *filep)
 106 -{
 107 -       char **file = (char **)filep;
 108 -
 109 -       if (status == 0)
 110 -               return;
 111 -       if (file == NULL || *file == NULL)
 112 -               return;
 113 -
 114 -       unlink(*file);
 115 +       efi_secdb_free(secdb);
 116  }
 117
 118  static void
 119 @@ -323,15 +310,6 @@ parse_input_files(list_t *infiles, char **outfile, efi_secdb_t **secdb,
 120         return status;
 121  }
 122
 123 -/*
 124 - * These need to be static globals so that they're not on main's stack when
 125 - * on_exit() fires.
 126 - */
 127 -static efi_secdb_t *secdb = NULL;
 128 -static list_t infiles;
 129 -static list_t actions;
 130 -static char *outfile = NULL;
 131 -
 132  int
 133  main(int argc, char *argv[])
 134  {
 135 @@ -351,6 +329,7 @@ main(int argc, char *argv[])
 136         bool do_sort_data = false;
 137         bool sort_descending = false;
 138         int status = 0;
 139 +       char *outfile = NULL;
 140
 141         const char sopts[] = ":aAc:dfg:h:i:Lo:rs:t:v?";
 142         const struct option lopts[] = {
 143 @@ -376,10 +355,9 @@ main(int argc, char *argv[])
 144         INIT_LIST_HEAD(&infiles);
 145         INIT_LIST_HEAD(&actions);
 146
 147 -       on_exit(free_actions, &actions);
 148 -       on_exit(free_infiles, &infiles);
 149 -       on_exit(maybe_free_secdb, &secdb);
 150 -       on_exit(maybe_do_unlink, &outfile);
 151 +       atexit(free_actions);
 152 +       atexit(free_infiles);
 153 +       atexit(maybe_free_secdb);
 154
 155         /*
 156          * parse the command line.
 157 @@ -587,24 +565,30 @@ main(int argc, char *argv[])
 158         outfd = open(outfile, flags, 0600);
 159         if (outfd < 0) {
 160                 char *tmpoutfile = outfile;
 161 -               if (errno == EEXIST)
 162 -                       outfile = NULL;
 163 +               if (errno != EEXIST)
 164 +                       unlink(outfile);
 165                 err(1, "could not open \"%s\"", tmpoutfile);
 166         }
 167
 168         rc = ftruncate(outfd, 0);
 169 -       if (rc < 0)
 170 +       if (rc < 0) {
 171 +               unlink(outfile);
 172                 err(1, "could not truncate output file \"%s\"", outfile);
 173 +       }
 174
 175         void *output;
 176         size_t size = 0;
 177         rc = efi_secdb_realize(secdb, &output, &size);
 178 -       if (rc < 0)
 179 +       if (rc < 0) {
 180 +               unlink(outfile);
 181                 secdb_err(1, "could not realize signature list");
 182 +       }
 183
 184         rc = write(outfd, output, size);
 185 -       if (rc < 0)
 186 +       if (rc < 0) {
 187 +               unlink(outfile);
 188                 err(1, "could not write signature list");
 189 +       }
 190
 191         close(outfd);
 192         xfree(output);
 193
 194 From df09b472419466987f2f30176dd00937e640aa9a Mon Sep 17 00:00:00 2001
 195 From: Natanael Copa <ncopa@alpinelinux.org>
 196 Date: Fri, 28 Jan 2022 12:29:00 +0100
 197 Subject: [PATCH 2/2] efisecdb: do not free optarg
 198
 199 The *outfile passed to parse_input_files can only be either set to
 200 optarg or be NULL. optarg should not be free'd and NULL does not need
 201 to.
 202
 203 Since we no longer use on_exit to unlink outfile we also don't need to
 204 set *outfile to NULL.
 205
 206 Fixes commit d91787035bc1 (efisecdb: add efisecdb)
 207
 208 Signed-off-by: Natanael Copa <ncopa@alpinelinux.org>
 209 ---
 210  src/efisecdb.c | 7 ++-----
 211  1 file changed, 2 insertions(+), 5 deletions(-)
 212
 213 diff --git a/src/efisecdb.c b/src/efisecdb.c
 214 index 6bd5ad90..70fa1847 100644
 215 --- a/src/efisecdb.c
 216 +++ b/src/efisecdb.c
 217 @@ -255,8 +255,7 @@ list_guids(void)
 218   * failure.
 219   */
 220  static int
 221 -parse_input_files(list_t *infiles, char **outfile, efi_secdb_t **secdb,
 222 -                 bool dump)
 223 +parse_input_files(list_t *infiles, efi_secdb_t **secdb, bool dump)
 224  {
 225         int status = 0;
 226         list_t *pos, *tmp;
 227 @@ -297,8 +296,6 @@ parse_input_files(list_t *infiles, char **outfile, efi_secdb_t **secdb,
 228                                 if (!dump)
 229                                         exit(1);
 230                                 status = 1;
 231 -                               xfree(*outfile);
 232 -                               *outfile = NULL;
 233                                 break;
 234                         }
 235                 }
 236 @@ -528,7 +525,7 @@ main(int argc, char *argv[])
 237         efi_secdb_set_bool(secdb, EFI_SECDB_SORT_DATA, do_sort_data);
 238         efi_secdb_set_bool(secdb, EFI_SECDB_SORT_DESCENDING, sort_descending);
 239
 240 -       status = parse_input_files(&infiles, &outfile, &secdb, dump);
 241 +       status = parse_input_files(&infiles, &secdb, dump);
 242         if (status == 0) {
 243                 for_each_action_safe(pos, tmp, &actions) {
 244                         action_t *action = list_entry(pos, action_t, list);