File: 100.iconv_hook-20040430-memleak.patch

package info (click to toggle)
libapache-mod-encoding 0.0.20021209-10
  • links: PTS
  • area: main
  • in suites: jessie, jessie-kfreebsd, lenny, squeeze, wheezy
  • size: 2,668 kB
  • ctags: 284
  • sloc: ansic: 94,392; sh: 13,778; makefile: 113
file content (47 lines) | stat: -rw-r--r-- 1,589 bytes parent folder | download | duplicates (4)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
diff -urN mod_encoding/lib/iconv_hook.c mod_encoding.20040430/lib/iconv_hook.c
--- mod_encoding/lib/iconv_hook.c	2002-06-08 18:19:01.000000000 +0900
+++ mod_encoding.20040430/lib/iconv_hook.c	2004-04-30 21:40:14.000000000 +0900
@@ -32,8 +32,9 @@
 }
 
 int
-iconv_hook_close(iconv_t cd) {
-  free((iconv_hook_t *)cd);
+iconv_hook_close(iconv_hook_t *cd) {
+  (*(cd->cm->iconv_close))(cd->cd);
+  free(cd);
   return 0;
 }
 
diff -urN mod_encoding/lib/iconv_hook_eucjp.c mod_encoding.20040430/lib/iconv_hook_eucjp.c
--- mod_encoding/lib/iconv_hook_eucjp.c	2002-06-10 22:57:52.000000000 +0900
+++ mod_encoding.20040430/lib/iconv_hook_eucjp.c	2004-04-30 21:40:14.000000000 +0900
@@ -111,6 +111,7 @@
         if (cl < 0xA0 || cl > 0xDF) {
           *srcbuf=(char *)(src-2);
           errno=EILSEQ;
+          free(tmpbuf);
           return -1;
         }
         *tmp++ = cl;
@@ -118,6 +119,7 @@
         /* We don't support JIS X 0212 */
         *srcbuf=(char *)(src-2);
         errno=EILSEQ;
+        free(tmpbuf);
         return -1;
       }
     }
diff -urN mod_encoding/lib/iconv_hook_ja_auto.c mod_encoding.20040430/lib/iconv_hook_ja_auto.c
--- mod_encoding/lib/iconv_hook_ja_auto.c	2002-06-08 18:32:41.000000000 +0900
+++ mod_encoding.20040430/lib/iconv_hook_ja_auto.c	2004-04-30 21:40:14.000000000 +0900
@@ -37,6 +37,9 @@
     cd->ienc = strdup(ienc);
 
     if (cd->oenc == NULL || cd->ienc == NULL){
+        if (cd->oenc == NULL) free (cd->oenc);
+        if (cd->ienc == NULL) free (cd->ienc);
+        free(cd);
 	return (iconv_t)(-1);
     } else {
   	return (iconv_t)cd;