Run memset only after having checked the return of malloc

Submitted by:	pluknet

diff --git a/usr.bin/localedef/collate.c b/usr.bin/localedef/collate.c
index 5a31a75bac5011d4e5cc2f0aa446b1b169c05cdc..3216d21dbcebbe9049501cb27cddb93da005020e 100644 (file)
--- a/usr.bin/localedef/collate.c
+++ b/usr.bin/localedef/collate.c
@@ -1256,11 +1256,11 @@ dump_collate(void)
         */
        RB_NUMNODES(collchar_t, collchars, &collchars, n);
        large = malloc(sizeof (collate_large_t) * n);
-       memset(large, 0, sizeof (collate_large_t) * n);
        if (large == NULL) {
                fprintf(stderr, "out of memory");
                return;
        }
+       memset(large, 0, sizeof (collate_large_t) * n);
 
        i = 0;
        RB_FOREACH(cc, collchars, &collchars) {