[cairo-commit] src/cairo-scaled-font.c

Bryce Harrington bryce at kemper.freedesktop.org
Sat Jun 16 05:15:27 UTC 2018


 src/cairo-scaled-font.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 61122127943dcaff41501e1b06df2a852c576e9b
Author: Bryce Harrington <bryce at bryceharrington.org>
Date:   Tue Jun 12 16:04:03 2018 -0700

    scaled-font: Fix glyph and cluster count checks (CID #983386)
    
    num_glyphs and num_clusters are explicitly checked to be non-NULL at the
    beginning of this routine, and by this point in the code both have been
    deref'd multiple times, so checking them for NULL here again is
    superfluous.
    
    It looks like the intent here is to verify the glyphs and clusters
    arrays are non-NULL unless their counts are zero, so change the tests
    accordingly.
    
    Coverity ID: #983386
    
    Signed-off-by: Bryce Harrington <bryce at bryceharrington.org>

diff --git a/src/cairo-scaled-font.c b/src/cairo-scaled-font.c
index f7a36c1df..8dff57d95 100644
--- a/src/cairo-scaled-font.c
+++ b/src/cairo-scaled-font.c
@@ -2052,7 +2052,7 @@ cairo_scaled_font_text_to_glyphs (cairo_scaled_font_t   *scaled_font,
 		    status = _cairo_error (CAIRO_STATUS_NEGATIVE_COUNT);
 		    goto DONE;
 		}
-		if (num_glyphs && *glyphs == NULL) {
+		if (*num_glyphs != 0 && *glyphs == NULL) {
 		    status = _cairo_error (CAIRO_STATUS_NULL_POINTER);
 		    goto DONE;
 		}
@@ -2062,7 +2062,7 @@ cairo_scaled_font_text_to_glyphs (cairo_scaled_font_t   *scaled_font,
 			status = _cairo_error (CAIRO_STATUS_NEGATIVE_COUNT);
 			goto DONE;
 		    }
-		    if (num_clusters && *clusters == NULL) {
+		    if (*num_clusters != 0 && *clusters == NULL) {
 			status = _cairo_error (CAIRO_STATUS_NULL_POINTER);
 			goto DONE;
 		    }


More information about the cairo-commit mailing list