From 83994913b6603c3a0dbb5ece4b3b5b47457ed99e Mon Sep 17 00:00:00 2001
From: Sebastian Huber <sebastian.huber@embedded-brains.de>
Date: Thu, 5 Mar 2020 21:48:10 +0100
Subject: imfs: Constify imfs_memfile_bytes_per_block

The CONFIGURE_IMFS_MEMFILE_BYTES_PER_BLOCK value is validated by
<rtems/confdefs/libio.h>.  Changing this value during runtime could lead
to memory corruption.

Update #3894.
---
 cpukit/libfs/src/imfs/imfs_initsupp.c  | 35 ----------------------------------
 cpukit/libfs/src/imfs/imfs_stat_file.c |  2 --
 2 files changed, 37 deletions(-)

(limited to 'cpukit/libfs')

diff --git a/cpukit/libfs/src/imfs/imfs_initsupp.c b/cpukit/libfs/src/imfs/imfs_initsupp.c
index 124a1622c9..58ec41f269 100644
--- a/cpukit/libfs/src/imfs/imfs_initsupp.c
+++ b/cpukit/libfs/src/imfs/imfs_initsupp.c
@@ -21,35 +21,6 @@
 #include <rtems/imfs.h>
 
 #include <errno.h>
-#include <stdlib.h>
-#include <string.h>
-#include <unistd.h>
-
-static int IMFS_determine_bytes_per_block(
-  int *dest_bytes_per_block,
-  int requested_bytes_per_block,
-  int default_bytes_per_block
-)
-{
-  bool is_valid = false;
-  int bit_mask;
-
-  /*
-   * check, whether requested bytes per block is valid
-   */
-  for (bit_mask = 16; !is_valid && (bit_mask <= 512); bit_mask <<= 1) {
-    if (bit_mask == requested_bytes_per_block) {
-      is_valid = true;
-      break;
-    }
-    if(bit_mask > requested_bytes_per_block)
-      break;
-  }
-  *dest_bytes_per_block = ((is_valid)
-			   ? requested_bytes_per_block
-			   : default_bytes_per_block);
-  return 0;
-}
 
 int IMFS_initialize_support(
   rtems_filesystem_mount_table_entry_t *mt_entry,
@@ -78,12 +49,6 @@ int IMFS_initialize_support(
   mt_entry->mt_fs_root->location.node_access = root_node;
   IMFS_Set_handlers( &mt_entry->mt_fs_root->location );
 
-  IMFS_determine_bytes_per_block(
-    &imfs_memfile_bytes_per_block,
-    imfs_rq_memfile_bytes_per_block,
-    IMFS_MEMFILE_DEFAULT_BYTES_PER_BLOCK
-  );
-
   return 0;
 }
 
diff --git a/cpukit/libfs/src/imfs/imfs_stat_file.c b/cpukit/libfs/src/imfs/imfs_stat_file.c
index 0135fe43e3..ef1d0095a6 100644
--- a/cpukit/libfs/src/imfs/imfs_stat_file.c
+++ b/cpukit/libfs/src/imfs/imfs_stat_file.c
@@ -19,8 +19,6 @@
 
 #include <rtems/imfs.h>
 
-int imfs_memfile_bytes_per_block;
-
 int IMFS_stat_file(
   const rtems_filesystem_location_info_t *loc,
   struct stat *buf
-- 
cgit v1.2.3