diff options
author | Owen Taylor <otaylor@redhat.com> | 2004-05-27 21:03:42 +0000 |
---|---|---|
committer | Owen Taylor <otaylor@src.gnome.org> | 2004-05-27 21:03:42 +0000 |
commit | 20dd3500dc55252c1b342ad9b4aab1f7ce53175e (patch) | |
tree | e314588f7f13d4cc5a431c99a353a2e0742baabd | |
parent | ff4b4e9e385b50974d8553a9fce6d71b801c3999 (diff) | |
download | pango-20dd3500dc55252c1b342ad9b4aab1f7ce53175e.tar.gz |
Free buffer->positions, clean up error returns that were returning
Thu May 27 16:57:30 2004 Owen Taylor <otaylor@redhat.com>
* pango/opentype/otlbuffer.c: Free buffer->positions,
clean up error returns that were returning uninitialized
values. (#139239, Behdad Esfahbod)
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | ChangeLog.pre-1-10 | 6 | ||||
-rw-r--r-- | ChangeLog.pre-1-6 | 6 | ||||
-rw-r--r-- | ChangeLog.pre-1-8 | 6 | ||||
-rw-r--r-- | pango/opentype/otlbuffer.c | 21 |
5 files changed, 37 insertions, 8 deletions
@@ -1,3 +1,9 @@ +Thu May 27 16:57:30 2004 Owen Taylor <otaylor@redhat.com> + + * pango/opentype/otlbuffer.c: Free buffer->positions, + clean up error returns that were returning uninitialized + values. (#139239, Behdad Esfahbod) + Thu May 27 15:42:20 2004 Owen Taylor <otaylor@redhat.com> * pango/pangox-fontmap.c pango/pangox.c: Add some missing diff --git a/ChangeLog.pre-1-10 b/ChangeLog.pre-1-10 index 4b7df6b1..736dd09d 100644 --- a/ChangeLog.pre-1-10 +++ b/ChangeLog.pre-1-10 @@ -1,3 +1,9 @@ +Thu May 27 16:57:30 2004 Owen Taylor <otaylor@redhat.com> + + * pango/opentype/otlbuffer.c: Free buffer->positions, + clean up error returns that were returning uninitialized + values. (#139239, Behdad Esfahbod) + Thu May 27 15:42:20 2004 Owen Taylor <otaylor@redhat.com> * pango/pangox-fontmap.c pango/pangox.c: Add some missing diff --git a/ChangeLog.pre-1-6 b/ChangeLog.pre-1-6 index 4b7df6b1..736dd09d 100644 --- a/ChangeLog.pre-1-6 +++ b/ChangeLog.pre-1-6 @@ -1,3 +1,9 @@ +Thu May 27 16:57:30 2004 Owen Taylor <otaylor@redhat.com> + + * pango/opentype/otlbuffer.c: Free buffer->positions, + clean up error returns that were returning uninitialized + values. (#139239, Behdad Esfahbod) + Thu May 27 15:42:20 2004 Owen Taylor <otaylor@redhat.com> * pango/pangox-fontmap.c pango/pangox.c: Add some missing diff --git a/ChangeLog.pre-1-8 b/ChangeLog.pre-1-8 index 4b7df6b1..736dd09d 100644 --- a/ChangeLog.pre-1-8 +++ b/ChangeLog.pre-1-8 @@ -1,3 +1,9 @@ +Thu May 27 16:57:30 2004 Owen Taylor <otaylor@redhat.com> + + * pango/opentype/otlbuffer.c: Free buffer->positions, + clean up error returns that were returning uninitialized + values. (#139239, Behdad Esfahbod) + Thu May 27 15:42:20 2004 Owen Taylor <otaylor@redhat.com> * pango/pangox-fontmap.c pango/pangox.c: Add some missing diff --git a/pango/opentype/otlbuffer.c b/pango/opentype/otlbuffer.c index b35a63fb..e9fd03b4 100644 --- a/pango/opentype/otlbuffer.c +++ b/pango/opentype/otlbuffer.c @@ -28,12 +28,15 @@ while (size > new_allocated) new_allocated += (new_allocated >> 1) + 8; - - if ( FT_REALLOC_ARRAY( buffer->in_string, buffer->allocated, new_allocated, OTL_GlyphItemRec ) ) + + error = FT_REALLOC_ARRAY( buffer->in_string, buffer->allocated, new_allocated, OTL_GlyphItemRec ); + if ( error ) return error; - if ( FT_REALLOC_ARRAY( buffer->out_string, buffer->allocated, new_allocated, OTL_GlyphItemRec ) ) + error = FT_REALLOC_ARRAY( buffer->out_string, buffer->allocated, new_allocated, OTL_GlyphItemRec ); + if ( error ) return error; - if ( FT_REALLOC_ARRAY( buffer->positions, buffer->allocated, new_allocated, OTL_PositionRec ) ) + error = FT_REALLOC_ARRAY( buffer->positions, buffer->allocated, new_allocated, OTL_PositionRec ); + if ( error ) return error; buffer->allocated = new_allocated; @@ -47,8 +50,9 @@ OTL_Buffer *buffer ) { FT_Error error; - - if ( FT_ALLOC( *buffer, sizeof( OTL_BufferRec ) ) ) + + error = FT_ALLOC( *buffer, sizeof( OTL_BufferRec ) ); + if ( error ) return error; (*buffer)->memory = memory; @@ -91,6 +95,7 @@ FT_FREE( buffer->in_string ); FT_FREE( buffer->out_string ); + FT_FREE( buffer->positions ); FT_FREE( buffer ); return FT_Err_Ok; @@ -117,7 +122,7 @@ OTL_GlyphItem glyph; error = otl_buffer_ensure( buffer, buffer->in_length + 1 ); - if ( error != FT_Err_Ok ) + if ( error ) return error; glyph = &buffer->in_string[buffer->in_length]; @@ -165,7 +170,7 @@ FT_UInt cluster; error = otl_buffer_ensure( buffer, buffer->out_pos + num_out ); - if ( error != FT_Err_Ok ) + if ( error ) return error; properties = buffer->in_string[buffer->in_pos].properties; |