fix possible buffer overflow

git-svn: trunk@1722
20 years ago · 337b3f5102
parent e362d35e3b
commit 337b3f5102
2 changed files with 5 additions and 1 deletions
--- a/clamav-devel/ChangeLog
+++ b/clamav-devel/ChangeLog
@ -1,3 +1,7 @@
+Fri Sep 16 16:49:14 CEST 2005 (tk)
+----------------------------------
+  * libclamav/upx.c: fix possible buffer overflow (acab)
+
 Fri Sep 16 16:39:37 CEST 2005 (tk)
 ----------------------------------
  * libclamav/fsg.c: fix possible infinite loop (acab)
--- a/clamav-devel/libclamav/upx.c
+++ b/clamav-devel/libclamav/upx.c
@ -117,7 +117,7 @@ int pefromupx (char *src, char *dst, int *dsize, uint32_t ep, uint32_t upx0, uin
  }
  
  sections = pehdr+0xf8;
-  if ( ! (sectcnt = pehdr[6]+256*pehdr[7])) {
+  if ( ! (sectcnt = (unsigned char)pehdr[6]+256*(unsigned char)pehdr[7])) {
    cli_dbgmsg("UPX: No sections? - giving up rebuild\n");
    return 0;
  }