Most uses of VAR_NAME_LEN within PSPP are wrong due to encoding issues:
the limit applies to variable names in the encoding used by the data
set, but most uses of VAR_NAME_LEN actually limit the length of a name
in UTF-8. The UTF-8 representation of a name can be longer or shorter
than its representation in the data set encoding, so it seems best to
eliminate references to VAR_NAME_LEN entirely.
/* PSPP - a program for statistical analysis.
/* PSPP - a program for statistical analysis.
- Copyright (C) 2007, 2008, 2009, 2010 Free Software Foundation, Inc.
+ Copyright (C) 2007, 2008, 2009, 2010, 2011 Free Software Foundation, Inc.
This program is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
This program is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
struct dictionary *dict = dict_create ();
struct file_handle *fh = NULL;
struct dfm_reader *reader = NULL;
struct dictionary *dict = dict_create ();
struct file_handle *fh = NULL;
struct dfm_reader *reader = NULL;
int record;
enum data_parser_type type;
int record;
enum data_parser_type type;
}
lex_match (lexer, T_EQUALS);
}
lex_match (lexer, T_EQUALS);
record = 1;
type = data_parser_get_type (parser);
do
{
record = 1;
type = data_parser_get_type (parser);
do
{
- char name[VAR_NAME_LEN + 1];
struct fmt_spec input, output;
struct fmt_spec input, output;
while (type == DP_FIXED && lex_match (lexer, T_SLASH))
{
while (type == DP_FIXED && lex_match (lexer, T_SLASH))
{
if (!lex_force_id (lexer))
goto error;
if (!lex_force_id (lexer))
goto error;
- strcpy (name, lex_tokcstr (lexer));
+ name = xstrdup (lex_tokcstr (lexer));
lex_get (lexer);
if (type == DP_DELIMITED)
lex_get (lexer);
if (type == DP_DELIMITED)
else
data_parser_add_fixed_field (parser, &input, var_get_case_index (v),
name, record, fc);
else
data_parser_add_fixed_field (parser, &input, var_get_case_index (v),
name, record, fc);
+ free (name);
+ name = NULL;
}
while (lex_token (lexer) != T_ENDCMD);
}
while (lex_token (lexer) != T_ENDCMD);
data_parser_destroy (parser);
dict_destroy (dict);
fh_unref (fh);
data_parser_destroy (parser);
dict_destroy (dict);
fh_unref (fh);
return CMD_CASCADING_FAILURE;
}
return CMD_CASCADING_FAILURE;
}