tftp: do not risk invoking Sorcerer's Apprentice syndrome

[oweals/busybox.git] / modutils / modutils.c

diff --git a/modutils/modutils.c b/modutils/modutils.c
index ef4f6191b6452b04b50673221aea0735f067d86c..0f6cb0f2db0c799a11fa41f649be2e2ffe90cd64 100644 (file)
--- a/modutils/modutils.c
+++ b/modutils/modutils.c
@@ -5,7 +5,6 @@
  *
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
-
 #include "modutils.h"
 
 #ifdef __UCLIBC__
@@ -17,19 +16,6 @@ extern int delete_module(const char *module, unsigned int flags);
 # define delete_module(mod, flags) syscall(__NR_delete_module, mod, flags)
 #endif
 
-/*
- a libbb candidate from ice age!
-*/
-llist_t FAST_FUNC *llist_find(llist_t *first, const char *str)
-{
-       while (first != NULL) {
-               if (strcmp(first->data, str) == 0)
-                       return first;
-               first = first->link;
-       }
-       return NULL;
-}
-
 void FAST_FUNC replace(char *s, char what, char with)
 {
        while (*s) {
@@ -111,9 +97,9 @@ char * FAST_FUNC parse_cmdline_module_options(char **argv)
 
 int FAST_FUNC bb_init_module(const char *filename, const char *options)
 {
-       size_t len = MAXINT(ssize_t);
+       size_t len;
        char *image;
-       int rc = ENOENT;
+       int rc;
 
        if (!options)
                options = "";
@@ -124,10 +110,12 @@ int FAST_FUNC bb_init_module(const char *filename, const char *options)
 #endif
 
        /* Use the 2.6 way */
+       len = INT_MAX - 4095;
+       rc = ENOENT;
        image = xmalloc_open_zipped_read_close(filename, &len);
        if (image) {
                rc = 0;
-               if (init_module(image, len, options ? options : "") != 0)
+               if (init_module(image, len, options) != 0)
                        rc = errno;
                free(image);
        }