contrib/lar: fix buffer overflow check, define os dir seperator
authorJo-Philipp Wich <jow@openwrt.org>
Mon, 6 Apr 2009 18:16:04 +0000 (18:16 +0000)
committerJo-Philipp Wich <jow@openwrt.org>
Mon, 6 Apr 2009 18:16:04 +0000 (18:16 +0000)
contrib/lar/lar.c
contrib/lar/lar.h

index ad6cfc8e5d4f492ee0d9545edd093c422aeae210..839b29299b3cd859cb319b47041dbb32294957a0 100644 (file)
@@ -183,7 +183,7 @@ lar_archive * lar_find_archive( const char *package )
 
        for( len = 0; package[len] != '\0'; len++ )
        {
-               if( len >= sizeof(buffer) )
+               if( len >= (sizeof(buffer) - 5) )
                        LAR_DIE("Package name exceeds maximum allowed length");
 
                if( package[len] == '.' ) seg++;
@@ -197,7 +197,7 @@ lar_archive * lar_find_archive( const char *package )
                                if( j < seg ) j++; else break;
                        }
 
-                       buffer[i] = ( package[i] == '.' ) ? '/' : package[i];
+                       buffer[i] = ( package[i] == '.' ) ? LAR_DIRSEP : package[i];
                }
 
                buffer[i+0] = '.'; buffer[i+1] = 'l'; buffer[i+2] = 'a';
@@ -219,7 +219,7 @@ lar_member * lar_find_member( lar_archive *ar, const char *package )
 
        for( len = 0; package[len] != '\0'; len++ )
        {
-               if( len >= sizeof(buffer) )
+               if( len >= (sizeof(buffer) - 5) )
                        LAR_DIE("Package name exceeds maximum allowed length");
 
                buffer[len] = ( package[len] == '.' ) ? '/' : package[len];
index c5b429b9ff42346766512fe4f372d97dc7cb2abd..59e6fa620975f91a6da8a337fa716c106dc001f7 100644 (file)
@@ -1,3 +1,6 @@
+#ifndef __LAR_H
+#define __LAR_H
+
 #include <stdio.h>
 #include <stdlib.h>
 #include <unistd.h>
@@ -25,6 +28,12 @@ int errno;
 #define LAR_FNAME_BUFFER 1024
 #define LAR_FNAME(s) char s[LAR_FNAME_BUFFER]
 
+#ifdef __WIN32__
+#define LAR_DIRSEP     '\\'
+#else
+#define LAR_DIRSEP     '/'
+#endif
+
 
 struct lar_index_item {
        uint32_t noffset;
@@ -76,3 +85,6 @@ int lar_close( lar_archive *ar );
 lar_archive * lar_find_archive( const char *package );
 
 lar_member * lar_find_member( lar_archive *ar, const char *package );
+
+#endif
+