Changeset 103db908 in mainline

Timestamp:

2018-11-22T10:44:22Z (5 years ago)

Author:

Maurizio Lombardi <mlombard@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

04c35fca

Parents:

dd500c26

Message:

fontviewer: draw(): fix memory leaks in the error code paths

File:

• uspace/app/fontviewer/fontviewer.c (modified) (3 diffs)

uspace/app/fontviewer/fontviewer.c

@@ -170 +170 @@
         source_t leading_bg = rgb(170, 238, 255);
         source_t leading_fg = rgb(0, 170, 212);
-
-        font_t *font;
+        font_t *info_font = NULL;
+        font_t *font = NULL;
+
         errno_t rc = create_font(&font, points);
         if (rc != EOK) {
                 printf("Failed creating font\n");
-                return rc;
-        }
-
-        font_t *info_font;
+                goto out_err;
+        }
+
         rc = embedded_font_create(&info_font, 16);
         if (rc != EOK) {
                 printf("Failed creating info font\n");
-                return rc;
+                goto out_err;
         }
 
@@ -188 +188 @@
         rc = font_get_metrics(font, &font_metrics);
         if (rc != EOK)
-                return rc;
+                goto out_err;
 
         surface_coord_t top = 50;
@@ -238 +238 @@
         }
 
-        font_release(font);
-        font_release(info_font);
-        return EOK;
+out_err:
+        if (font)
+                font_release(font);
+        if (info_font)
+                font_release(info_font);
+        return rc;
 }