- Check parameters before allocating memory
authorGregor Schneider <grschneider@gmail.com>
Tue, 25 Nov 2008 12:53:21 +0000 (12:53 +0000)
committerGregor Schneider <grschneider@gmail.com>
Tue, 25 Nov 2008 12:53:21 +0000 (12:53 +0000)
- Patch by Daniel Zimmermann, <netzimme AT aim DOT com>
- Fixes possible memory leaks pointed out by CID 605, bug #3910

svn path=/trunk/; revision=37628

reactos/lib/sdk/crt/time/strftime.c

index c45ad2c..6321928 100644 (file)
@@ -233,6 +233,8 @@ size_t wcsftime(wchar_t* s, size_t maxsize, const wchar_t* format, const struct
   char *x;
   char *f;
   size_t i,j;
+  if ((gsize = maxsize) < 1)
+    return 0;
   x = malloc(maxsize);
   j = wcslen(format);
   f = malloc(j+1);
@@ -240,8 +242,6 @@ size_t wcsftime(wchar_t* s, size_t maxsize, const wchar_t* format, const struct
        f[i] = (char)*format;
   f[i] = 0;
   pt = x;
-  if ((gsize = maxsize) < 1)
-    return 0;
   if (_fmt(f, t)) {
     *pt = '\0';
     free(f);