[PATCH:libfontenc 1/2] Replace malloc(strlen()) + strcpy() with strdup()
Jamey Sharp
jamey at minilop.net
Sat May 15 14:22:06 PDT 2010
For both patches:
Reviewed-by: Jamey Sharp <jamey at minilop.net>
On Fri, May 14, 2010 at 8:18 PM, Alan Coopersmith
<alan.coopersmith at oracle.com> wrote:
> Now that we use malloc directly instead of xalloc, we can use
> strdup directly too.
>
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
> src/encparse.c | 12 ++++--------
> src/fontenc.c | 3 +--
> 2 files changed, 5 insertions(+), 10 deletions(-)
>
> diff --git a/src/encparse.c b/src/encparse.c
> index b288c8c..33e0a05 100644
> --- a/src/encparse.c
> +++ b/src/encparse.c
> @@ -488,10 +488,9 @@ parseEncodingFile(FontFilePtr f, int headerOnly)
> encoding = malloc(sizeof(FontEncRec));
> if(encoding == NULL)
> goto error;
> - encoding->name = malloc(strlen(keyword_value)+1);
> + encoding->name = strdup(keyword_value);
> if(encoding->name == NULL)
> goto error;
> - strcpy(encoding->name, keyword_value);
> encoding->size = 256;
> encoding->row_size = 0;
> encoding->mappings = NULL;
> @@ -508,10 +507,9 @@ parseEncodingFile(FontFilePtr f, int headerOnly)
> case EOF_LINE: goto done;
> case ALIAS_LINE:
> if(numaliases < MAXALIASES) {
> - aliases[numaliases] = malloc(strlen(keyword_value)+1);
> + aliases[numaliases] = strdup(keyword_value);
> if(aliases[numaliases] == NULL)
> goto error;
> - strcpy(aliases[numaliases], keyword_value);
> numaliases++;
> }
> goto no_mapping;
> @@ -718,11 +716,10 @@ parseEncodingFile(FontFilePtr f, int headerOnly)
> nam[i]=NULL;
> last = value1;
> }
> - nam[value1] = malloc(strlen(keyword_value)+1);
> + nam[value1] = strdup(keyword_value);
> if(nam[value1] == NULL) {
> goto error;
> }
> - strcpy(nam[value1], keyword_value);
> goto string_mapping;
>
> default: goto string_mapping; /* ignore unknown lines */
> @@ -781,10 +778,9 @@ FontEncDirectory(void)
> if(dir == NULL) {
> char *c = getenv("FONT_ENCODINGS_DIRECTORY");
> if(c) {
> - dir = malloc(strlen(c) + 1);
> + dir = strdup(c);
> if(!dir)
> return NULL;
> - strcpy(dir, c);
> } else {
> dir = FONT_ENCODINGS_DIRECTORY;
> }
> diff --git a/src/fontenc.c b/src/fontenc.c
> index 1a24fc2..8718eff 100644
> --- a/src/fontenc.c
> +++ b/src/fontenc.c
> @@ -753,10 +753,9 @@ FontEncLoad(const char *encoding_name, const char *filename)
> char *new_name;
> int numaliases = 0;
>
> - new_name = malloc(strlen(encoding_name) + 1);
> + new_name = strdup(encoding_name);
> if(new_name == NULL)
> return NULL;
> - strcpy(new_name, encoding_name);
> if(encoding->aliases) {
> for(alias = encoding->aliases; *alias; alias++)
> numaliases++;
> --
> 1.5.6.5
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
More information about the xorg-devel
mailing list