Remove useless checks for NULL before freeing

This patch has been generated by the following Coccinelle semantic patch:
// Remove useless checks for NULL before freeing
//
// free (NULL) is a no-op, so there is no need to avoid it

@@
expression E;
@@
+ free (E);
+ E = NULL;
- if (unlikely (E != NULL)) {
-   free(E);
(
-   E = NULL;
|
-   E = 0;
)
   ...
- }

@@
expression E;
@@
+ free (E);
- if (unlikely (E != NULL)) {
-   free (E);
- }

1 files changed, 3 insertions, 6 deletions

diff --git a/src/cairo-type1-subset.c b/src/cairo-type1-subset.c
index 6c94a83f9..a637e24f3 100644
--- a/src/cairo-type1-subset.c
+++ b/src/cairo-type1-subset.c
@@ -1492,17 +1492,14 @@ _cairo_type1_font_subset_fini (cairo_type1_font_subset_t *font)
     _cairo_array_fini (&font->glyph_names_array);
     _cairo_array_fini (&font->glyphs_array);
 
-    if (font->subrs)
-	free (font->subrs);
+    free (font->subrs);
 
     if (font->output != NULL)
 	status = _cairo_output_stream_destroy (font->output);
 
-    if (font->base.base_font)
-	free (font->base.base_font);
+    free (font->base.base_font);
 
-    if (font->subset_index_to_glyphs)
-	free (font->subset_index_to_glyphs);
+    free (font->subset_index_to_glyphs);
 
     return status;
 }