git.tremily.us Git - gentoo.git/blob

   1 From 9d25ca61603a2eed08d1b58b25c81f878c729474 Mon Sep 17 00:00:00 2001
   2 From: James Carter <jwcart2@tycho.nsa.gov>
   3 Date: Mon, 3 Oct 2016 14:15:40 -0400
   4 Subject: [PATCH] libsemanage: Fixes bug preventing the installation of base
   5  modules
   6
   7 Commit 7a728e46 changed module installation so that a module pp would
   8 be installed using its module name instead of its filename and a warning
   9 would be printed if they were different. With this change, base modules
  10 could no longer be installed because of the way error handling was done.
  11
  12 This change fixes the error handling, so that when a base module is
  13 installed it will be installed using its filename (since it does not
  14 have a module name).
  15
  16 Based on bug report by Jason Zaman
  17
  18 Signed-off-by: James Carter <jwcart2@tycho.nsa.gov>
  19 ---
  20  libsemanage/src/direct_api.c | 39 ++++++++++++++-------------------------
  21  1 file changed, 14 insertions(+), 25 deletions(-)
  22
  23 diff --git a/libsemanage/src/direct_api.c b/libsemanage/src/direct_api.c
  24 index 3719cb1..e5c72cd 100644
  25 --- libsemanage/src/direct_api.c
  26 +++ libsemanage/src/direct_api.c
  27 @@ -368,7 +368,7 @@ static int semanage_direct_begintrans(semanage_handle_t * sh)
  28   * 'version' to module's version.  The caller is responsible for
  29   * free()ing 'module_name', and 'version'; they will be
  30   * set to NULL upon entering this function.  Returns 0 on success, -1
  31 - * if out of memory, or -2 if data did not represent a module.
  32 + * if out of memory.
  33   */
  34  static int parse_module_headers(semanage_handle_t * sh, char *module_data,
  35                                 size_t data_len, char **module_name,
  36 @@ -384,23 +384,10 @@ static int parse_module_headers(semanage_handle_t * sh, char *module_data,
  37         }
  38         sepol_policy_file_set_mem(pf, module_data, data_len);
  39         sepol_policy_file_set_handle(pf, sh->sepolh);
  40 -       if (module_data == NULL ||
  41 -           data_len == 0 ||
  42 +       if (module_data != NULL && data_len > 0)
  43             sepol_module_package_info(pf, &file_type, module_name,
  44 -                                     version) == -1) {
  45 -               sepol_policy_file_free(pf);
  46 -               ERR(sh, "Could not parse module data.");
  47 -               return -2;
  48 -       }
  49 +                                     version);
  50         sepol_policy_file_free(pf);
  51 -       if (file_type != SEPOL_POLICY_MOD) {
  52 -               if (file_type == SEPOL_POLICY_BASE)
  53 -                       ERR(sh,
  54 -                           "Received a base module, expected a non-base module.");
  55 -               else
  56 -                       ERR(sh, "Data did not represent a module.");
  57 -               return -2;
  58 -       }
  59
  60         return 0;
  61  }
  62 @@ -1608,22 +1595,24 @@ static int semanage_direct_install_file(semanage_handle_t * sh,
  63                 lang_ext = separator + 1;
  64         }
  65
  66 -       if (strcmp(lang_ext, "pp") != 0) {
  67 +       if (strcmp(lang_ext, "pp") == 0) {
  68 +               retval = parse_module_headers(sh, data, data_len, &module_name, &version);
  69 +               free(version);
  70 +               if (retval != 0)
  71 +                       goto cleanup;
  72 +       }
  73 +
  74 +       if (module_name == NULL) {
  75                 module_name = strdup(filename);
  76                 if (module_name == NULL) {
  77                         ERR(sh, "No memory available for module_name.\n");
  78                         retval = -1;
  79                         goto cleanup;
  80                 }
  81 -       } else {
  82 -               if ((retval = parse_module_headers(sh, data, data_len, &module_name, &version)) != 0)
  83 -                       goto cleanup;
  84 -
  85 -               if (strcmp(module_name, filename) != 0)
  86 -                       fprintf(stderr, "Warning: SELinux userspace will refer to the module from %s as %s rather than %s\n", install_filename, module_name, filename);
  87 -
  88 -               free(version);
  89 +       } else if (strcmp(module_name, filename) != 0) {
  90 +               fprintf(stderr, "Warning: SELinux userspace will refer to the module from %s as %s rather than %s\n", install_filename, module_name, filename);
  91         }
  92 +
  93         retval = semanage_direct_install(sh, data, data_len, module_name, lang_ext);
  94
  95  cleanup:
  96 --
  97 2.7.3
  98