From: Brian Aker Date: Thu, 24 Nov 2011 03:55:53 +0000 (-0800) Subject: Fixes for returning better error in parser if namespace is invoked twice. X-Git-Tag: 1.0.3~4^2~3^2~6 X-Git-Url: https://git.m6w6.name/?a=commitdiff_plain;h=1825425e12755baebb754a9c4a4df69e3cea0bc9;p=awesomized%2Flibmemcached Fixes for returning better error in parser if namespace is invoked twice. --- diff --git a/Makefile.am b/Makefile.am index d19991ec..b9e758b0 100644 --- a/Makefile.am +++ b/Makefile.am @@ -26,8 +26,6 @@ EXTRA_DIST= \ config/pandora-plugin \ config/uncrustify.cfg -libmemcached_libmemcached_la_LDFLAGS= - include libtest/include.am include libhashkit-1.0/include.am diff --git a/libmemcached-1.0/return.h b/libmemcached-1.0/return.h index fa36243f..e5788c50 100644 --- a/libmemcached-1.0/return.h +++ b/libmemcached-1.0/return.h @@ -85,6 +85,7 @@ enum memcached_return_t { MEMCACHED_DEPRECATED, MEMCACHED_IN_PROGRESS, MEMCACHED_SERVER_TEMPORARILY_DISABLED, + MEMCACHED_SERVER_MEMORY_ALLOCATION_FAILURE, MEMCACHED_MAXIMUM_RETURN /* Always add new error code before */ }; diff --git a/libmemcached/callback.cc b/libmemcached/callback.cc index 83d8ce8f..36f4b309 100644 --- a/libmemcached/callback.cc +++ b/libmemcached/callback.cc @@ -90,7 +90,6 @@ void *memcached_callback_get(memcached_st *ptr, memcached_return_t *error) { memcached_return_t local_error; - if (error == NULL) { error = &local_error; diff --git a/libmemcached/csl/context.cc b/libmemcached/csl/context.cc index 1a60fede..36ff2587 100644 --- a/libmemcached/csl/context.cc +++ b/libmemcached/csl/context.cc @@ -42,15 +42,14 @@ void Context::abort(const char *error_arg, yytokentype last_token, const char *l { rc= MEMCACHED_PARSE_ERROR; (void)last_token; - (void)last_token_str; if (error_arg) { - memcached_set_parser_error(*memc, MEMCACHED_AT, "Error occured while parsing: %s", error_arg); + memcached_set_parser_error(*memc, MEMCACHED_AT, "%s", error_arg); return; } - memcached_set_parser_error(*memc, MEMCACHED_AT, "Error occured while parsing: %s", memcached_strerror(NULL, MEMCACHED_PARSE_ERROR)); + memcached_set_parser_error(*memc, MEMCACHED_AT, "%s", last_token_str ? last_token_str : " "); } void Context::error(const char *error_arg, yytokentype last_token, const char *last_token_str) diff --git a/libmemcached/csl/parser.cc b/libmemcached/csl/parser.cc index 63d92e8e..855b6fdd 100644 --- a/libmemcached/csl/parser.cc +++ b/libmemcached/csl/parser.cc @@ -1,9 +1,8 @@ -/* A Bison parser, made by GNU Bison 2.4.3. */ +/* A Bison parser, made by GNU Bison 2.5. */ -/* Skeleton implementation for Bison's Yacc-like parsers in C +/* Bison implementation for Yacc-like parsers in C - Copyright (C) 1984, 1989, 1990, 2000, 2001, 2002, 2003, 2004, 2005, 2006, - 2009, 2010 Free Software Foundation, Inc. + Copyright (C) 1984, 1989-1990, 2000-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 @@ -45,7 +44,7 @@ #define YYBISON 1 /* Bison version. */ -#define YYBISON_VERSION "2.4.3" +#define YYBISON_VERSION "2.5" /* Skeleton name. */ #define YYSKELETON_NAME "yacc.c" @@ -74,7 +73,7 @@ /* Copy the first part of user declarations. */ -/* Line 189 of yacc.c */ +/* Line 268 of yacc.c */ #line 36 "libmemcached/csl/parser.yy" @@ -115,8 +114,8 @@ inline void __config_error(Context *context, yyscan_t *scanner, const char *erro -/* Line 189 of yacc.c */ -#line 120 "libmemcached/csl/parser.cc" +/* Line 268 of yacc.c */ +#line 119 "libmemcached/csl/parser.cc" /* Enabling traces. */ #ifndef YYDEBUG @@ -227,8 +226,8 @@ inline void __config_error(Context *context, yyscan_t *scanner, const char *erro /* Copy the second part of user declarations. */ -/* Line 264 of yacc.c */ -#line 232 "libmemcached/csl/parser.cc" +/* Line 343 of yacc.c */ +#line 231 "libmemcached/csl/parser.cc" #ifdef short # undef short @@ -331,11 +330,11 @@ YYID (yyi) # define alloca _alloca # else # define YYSTACK_ALLOC alloca -# if ! defined _ALLOCA_H && ! defined _STDLIB_H && (defined __STDC__ || defined __C99__FUNC__ \ +# if ! defined _ALLOCA_H && ! defined EXIT_SUCCESS && (defined __STDC__ || defined __C99__FUNC__ \ || defined __cplusplus || defined _MSC_VER) # include /* INFRINGES ON USER NAME SPACE */ -# ifndef _STDLIB_H -# define _STDLIB_H 1 +# ifndef EXIT_SUCCESS +# define EXIT_SUCCESS 0 # endif # endif # endif @@ -358,24 +357,24 @@ YYID (yyi) # ifndef YYSTACK_ALLOC_MAXIMUM # define YYSTACK_ALLOC_MAXIMUM YYSIZE_MAXIMUM # endif -# if (defined __cplusplus && ! defined _STDLIB_H \ +# if (defined __cplusplus && ! defined EXIT_SUCCESS \ && ! ((defined YYMALLOC || defined malloc) \ && (defined YYFREE || defined free))) # include /* INFRINGES ON USER NAME SPACE */ -# ifndef _STDLIB_H -# define _STDLIB_H 1 +# ifndef EXIT_SUCCESS +# define EXIT_SUCCESS 0 # endif # endif # ifndef YYMALLOC # define YYMALLOC malloc -# if ! defined malloc && ! defined _STDLIB_H && (defined __STDC__ || defined __C99__FUNC__ \ +# if ! defined malloc && ! defined EXIT_SUCCESS && (defined __STDC__ || defined __C99__FUNC__ \ || defined __cplusplus || defined _MSC_VER) void *malloc (YYSIZE_T); /* INFRINGES ON USER NAME SPACE */ # endif # endif # ifndef YYFREE # define YYFREE free -# if ! defined free && ! defined _STDLIB_H && (defined __STDC__ || defined __C99__FUNC__ \ +# if ! defined free && ! defined EXIT_SUCCESS && (defined __STDC__ || defined __C99__FUNC__ \ || defined __cplusplus || defined _MSC_VER) void free (void *); /* INFRINGES ON USER NAME SPACE */ # endif @@ -404,23 +403,7 @@ union yyalloc ((N) * (sizeof (yytype_int16) + sizeof (YYSTYPE)) \ + YYSTACK_GAP_MAXIMUM) -/* Copy COUNT objects from FROM to TO. The source and destination do - not overlap. */ -# ifndef YYCOPY -# if defined __GNUC__ && 1 < __GNUC__ -# define YYCOPY(To, From, Count) \ - __builtin_memcpy (To, From, (Count) * sizeof (*(From))) -# else -# define YYCOPY(To, From, Count) \ - do \ - { \ - YYSIZE_T yyi; \ - for (yyi = 0; yyi < (Count); yyi++) \ - (To)[yyi] = (From)[yyi]; \ - } \ - while (YYID (0)) -# endif -# endif +# define YYCOPY_NEEDED 1 /* Relocate STACK from its old location to the new one. The local variables YYSIZE and YYSTACKSIZE give the old and new number of @@ -440,6 +423,26 @@ union yyalloc #endif +#if defined YYCOPY_NEEDED && YYCOPY_NEEDED +/* Copy COUNT objects from FROM to TO. The source and destination do + not overlap. */ +# ifndef YYCOPY +# if defined __GNUC__ && 1 < __GNUC__ +# define YYCOPY(To, From, Count) \ + __builtin_memcpy (To, From, (Count) * sizeof (*(From))) +# else +# define YYCOPY(To, From, Count) \ + do \ + { \ + YYSIZE_T yyi; \ + for (yyi = 0; yyi < (Count); yyi++) \ + (To)[yyi] = (From)[yyi]; \ + } \ + while (YYID (0)) +# endif +# endif +#endif /* !YYCOPY_NEEDED */ + /* YYFINAL -- State number of the termination state. */ #define YYFINAL 71 /* YYLAST -- Last index in YYTABLE. */ @@ -539,11 +542,11 @@ static const yytype_uint16 yyrline[] = { 0, 172, 172, 173, 177, 179, 181, 183, 188, 193, 197, 201, 212, 220, 228, 235, 239, 243, 247, 251, - 258, 265, 276, 283, 290, 297, 303, 307, 311, 315, - 319, 323, 327, 331, 335, 339, 343, 347, 354, 358, - 362, 366, 370, 374, 378, 382, 386, 390, 394, 398, - 405, 406, 411, 412, 417, 421, 425, 429, 433, 437, - 441, 445, 449, 456, 460, 467, 471, 475 + 263, 270, 281, 288, 295, 302, 308, 312, 316, 320, + 324, 328, 332, 336, 340, 344, 348, 352, 359, 363, + 367, 371, 375, 379, 383, 387, 391, 395, 399, 403, + 410, 411, 416, 417, 422, 426, 430, 434, 438, 442, + 446, 450, 454, 461, 465, 472, 476, 480 }; #endif @@ -613,8 +616,8 @@ static const yytype_uint8 yyr2[] = 1, 1, 1, 1, 1, 1, 1, 1 }; -/* YYDEFACT[STATE-NAME] -- Default rule to reduce with in state - STATE-NUM when YYTABLE doesn't specify something else to do. Zero +/* YYDEFACT[STATE-NAME] -- Default reduction number in state STATE-NUM. + Performed when YYTABLE doesn't specify something else to do. Zero means the default is an error. */ static const yytype_uint8 yydefact[] = { @@ -661,8 +664,7 @@ static const yytype_int8 yypgoto[] = /* YYTABLE[YYPACT[STATE-NUM]]. What to do in state STATE-NUM. If positive, shift that token. If negative, reduce the rule which - number is the opposite. If zero, do what YYDEFACT says. - If YYTABLE_NINF, syntax error. */ + number is the opposite. If YYTABLE_NINF, syntax error. */ #define YYTABLE_NINF -1 static const yytype_uint8 yytable[] = { @@ -676,6 +678,12 @@ static const yytype_uint8 yytable[] = 66, 84, 0, 0, 72 }; +#define yypact_value_is_default(yystate) \ + ((yystate) == (-61)) + +#define yytable_value_is_error(yytable_value) \ + YYID (0) + static const yytype_int8 yycheck[] = { 0, 3, 4, 5, 6, 7, 8, 9, 10, 11, @@ -736,7 +744,6 @@ do \ { \ yychar = (Token); \ yylval = (Value); \ - yytoken = YYTRANSLATE (yychar); \ YYPOPSTACK (1); \ goto yybackup; \ } \ @@ -778,19 +785,10 @@ while (YYID (0)) #endif -/* YY_LOCATION_PRINT -- Print the location on the stream. - This macro was not mandated originally: define only if we know - we won't break user code: when these are the locations we know. */ +/* This macro is provided for backward compatibility. */ #ifndef YY_LOCATION_PRINT -# if defined YYLTYPE_IS_TRIVIAL && YYLTYPE_IS_TRIVIAL -# define YY_LOCATION_PRINT(File, Loc) \ - fprintf (File, "%d.%d-%d.%d", \ - (Loc).first_line, (Loc).first_column, \ - (Loc).last_line, (Loc).last_column) -# else -# define YY_LOCATION_PRINT(File, Loc) ((void) 0) -# endif +# define YY_LOCATION_PRINT(File, Loc) ((void) 0) #endif @@ -990,7 +988,6 @@ int yydebug; # define YYMAXDEPTH 10000 #endif - #if YYERROR_VERBOSE @@ -1093,115 +1090,142 @@ yytnamerr (char *yyres, const char *yystr) } # endif -/* Copy into YYRESULT an error message about the unexpected token - YYCHAR while in state YYSTATE. Return the number of bytes copied, - including the terminating null byte. If YYRESULT is null, do not - copy anything; just return the number of bytes that would be - copied. As a special case, return 0 if an ordinary "syntax error" - message will do. Return YYSIZE_MAXIMUM if overflow occurs during - size calculation. */ -static YYSIZE_T -yysyntax_error (char *yyresult, int yystate, int yychar) -{ - int yyn = yypact[yystate]; +/* Copy into *YYMSG, which is of size *YYMSG_ALLOC, an error message + about the unexpected token YYTOKEN for the state stack whose top is + YYSSP. - if (! (YYPACT_NINF < yyn && yyn <= YYLAST)) - return 0; - else - { - int yytype = YYTRANSLATE (yychar); - YYSIZE_T yysize0 = yytnamerr (0, yytname[yytype]); - YYSIZE_T yysize = yysize0; - YYSIZE_T yysize1; - int yysize_overflow = 0; - enum { YYERROR_VERBOSE_ARGS_MAXIMUM = 5 }; - char const *yyarg[YYERROR_VERBOSE_ARGS_MAXIMUM]; - int yyx; - -# if 0 - /* This is so xgettext sees the translatable formats that are - constructed on the fly. */ - YY_("syntax error, unexpected %s"); - YY_("syntax error, unexpected %s, expecting %s"); - YY_("syntax error, unexpected %s, expecting %s or %s"); - YY_("syntax error, unexpected %s, expecting %s or %s or %s"); - YY_("syntax error, unexpected %s, expecting %s or %s or %s or %s"); -# endif - char *yyfmt; - char const *yyf; - static char const yyunexpected[] = "syntax error, unexpected %s"; - static char const yyexpecting[] = ", expecting %s"; - static char const yyor[] = " or %s"; - char yyformat[sizeof yyunexpected - + sizeof yyexpecting - 1 - + ((YYERROR_VERBOSE_ARGS_MAXIMUM - 2) - * (sizeof yyor - 1))]; - char const *yyprefix = yyexpecting; - - /* Start YYX at -YYN if negative to avoid negative indexes in - YYCHECK. */ - int yyxbegin = yyn < 0 ? -yyn : 0; - - /* Stay within bounds of both yycheck and yytname. */ - int yychecklim = YYLAST - yyn + 1; - int yyxend = yychecklim < YYNTOKENS ? yychecklim : YYNTOKENS; - int yycount = 1; - - yyarg[0] = yytname[yytype]; - yyfmt = yystpcpy (yyformat, yyunexpected); - - for (yyx = yyxbegin; yyx < yyxend; ++yyx) - if (yycheck[yyx + yyn] == yyx && yyx != YYTERROR) - { - if (yycount == YYERROR_VERBOSE_ARGS_MAXIMUM) - { - yycount = 1; - yysize = yysize0; - yyformat[sizeof yyunexpected - 1] = '\0'; - break; - } - yyarg[yycount++] = yytname[yyx]; - yysize1 = yysize + yytnamerr (0, yytname[yyx]); - yysize_overflow |= (yysize1 < yysize); - yysize = yysize1; - yyfmt = yystpcpy (yyfmt, yyprefix); - yyprefix = yyor; - } + Return 0 if *YYMSG was successfully written. Return 1 if *YYMSG is + not large enough to hold the message. In that case, also set + *YYMSG_ALLOC to the required number of bytes. Return 2 if the + required number of bytes is too large to store. */ +static int +yysyntax_error (YYSIZE_T *yymsg_alloc, char **yymsg, + yytype_int16 *yyssp, int yytoken) +{ + YYSIZE_T yysize0 = yytnamerr (0, yytname[yytoken]); + YYSIZE_T yysize = yysize0; + YYSIZE_T yysize1; + enum { YYERROR_VERBOSE_ARGS_MAXIMUM = 5 }; + /* Internationalized format string. */ + const char *yyformat = 0; + /* Arguments of yyformat. */ + char const *yyarg[YYERROR_VERBOSE_ARGS_MAXIMUM]; + /* Number of reported tokens (one for the "unexpected", one per + "expected"). */ + int yycount = 0; + + /* There are many possibilities here to consider: + - Assume YYFAIL is not used. It's too flawed to consider. See + + for details. YYERROR is fine as it does not invoke this + function. + - If this state is a consistent state with a default action, then + the only way this function was invoked is if the default action + is an error action. In that case, don't check for expected + tokens because there are none. + - The only way there can be no lookahead present (in yychar) is if + this state is a consistent state with a default action. Thus, + detecting the absence of a lookahead is sufficient to determine + that there is no unexpected or expected token to report. In that + case, just report a simple "syntax error". + - Don't assume there isn't a lookahead just because this state is a + consistent state with a default action. There might have been a + previous inconsistent state, consistent state with a non-default + action, or user semantic action that manipulated yychar. + - Of course, the expected token list depends on states to have + correct lookahead information, and it depends on the parser not + to perform extra reductions after fetching a lookahead from the + scanner and before detecting a syntax error. Thus, state merging + (from LALR or IELR) and default reductions corrupt the expected + token list. However, the list is correct for canonical LR with + one exception: it will still contain any token that will not be + accepted due to an error action in a later state. + */ + if (yytoken != YYEMPTY) + { + int yyn = yypact[*yyssp]; + yyarg[yycount++] = yytname[yytoken]; + if (!yypact_value_is_default (yyn)) + { + /* Start YYX at -YYN if negative to avoid negative indexes in + YYCHECK. In other words, skip the first -YYN actions for + this state because they are default actions. */ + int yyxbegin = yyn < 0 ? -yyn : 0; + /* Stay within bounds of both yycheck and yytname. */ + int yychecklim = YYLAST - yyn + 1; + int yyxend = yychecklim < YYNTOKENS ? yychecklim : YYNTOKENS; + int yyx; + + for (yyx = yyxbegin; yyx < yyxend; ++yyx) + if (yycheck[yyx + yyn] == yyx && yyx != YYTERROR + && !yytable_value_is_error (yytable[yyx + yyn])) + { + if (yycount == YYERROR_VERBOSE_ARGS_MAXIMUM) + { + yycount = 1; + yysize = yysize0; + break; + } + yyarg[yycount++] = yytname[yyx]; + yysize1 = yysize + yytnamerr (0, yytname[yyx]); + if (! (yysize <= yysize1 + && yysize1 <= YYSTACK_ALLOC_MAXIMUM)) + return 2; + yysize = yysize1; + } + } + } - yyf = YY_(yyformat); - yysize1 = yysize + yystrlen (yyf); - yysize_overflow |= (yysize1 < yysize); - yysize = yysize1; + switch (yycount) + { +# define YYCASE_(N, S) \ + case N: \ + yyformat = S; \ + break + YYCASE_(0, YY_("syntax error")); + YYCASE_(1, YY_("syntax error, unexpected %s")); + YYCASE_(2, YY_("syntax error, unexpected %s, expecting %s")); + YYCASE_(3, YY_("syntax error, unexpected %s, expecting %s or %s")); + YYCASE_(4, YY_("syntax error, unexpected %s, expecting %s or %s or %s")); + YYCASE_(5, YY_("syntax error, unexpected %s, expecting %s or %s or %s or %s")); +# undef YYCASE_ + } - if (yysize_overflow) - return YYSIZE_MAXIMUM; + yysize1 = yysize + yystrlen (yyformat); + if (! (yysize <= yysize1 && yysize1 <= YYSTACK_ALLOC_MAXIMUM)) + return 2; + yysize = yysize1; - if (yyresult) - { - /* Avoid sprintf, as that infringes on the user's name space. - Don't have undefined behavior even if the translation - produced a string with the wrong number of "%s"s. */ - char *yyp = yyresult; - int yyi = 0; - while ((*yyp = *yyf) != '\0') - { - if (*yyp == '%' && yyf[1] == 's' && yyi < yycount) - { - yyp += yytnamerr (yyp, yyarg[yyi++]); - yyf += 2; - } - else - { - yyp++; - yyf++; - } - } - } - return yysize; + if (*yymsg_alloc < yysize) + { + *yymsg_alloc = 2 * yysize; + if (! (yysize <= *yymsg_alloc + && *yymsg_alloc <= YYSTACK_ALLOC_MAXIMUM)) + *yymsg_alloc = YYSTACK_ALLOC_MAXIMUM; + return 1; } + + /* Avoid sprintf, as that infringes on the user's name space. + Don't have undefined behavior even if the translation + produced a string with the wrong number of "%s"s. */ + { + char *yyp = *yymsg; + int yyi = 0; + while ((*yyp = *yyformat) != '\0') + if (*yyp == '%' && yyformat[1] == 's' && yyi < yycount) + { + yyp += yytnamerr (yyp, yyarg[yyi++]); + yyformat += 2; + } + else + { + yyp++; + yyformat++; + } + } + return 0; } #endif /* YYERROR_VERBOSE */ - /*-----------------------------------------------. | Release the memory associated to this symbol. | @@ -1238,6 +1262,7 @@ yydestruct (yymsg, yytype, yyvaluep, context, scanner) } } + /* Prevent warnings from -Wmissing-prototypes. */ #ifdef YYPARSE_PARAM #if defined __STDC__ || defined __cplusplus @@ -1254,12 +1279,9 @@ int yyparse (); #endif /* ! YYPARSE_PARAM */ - - - -/*-------------------------. -| yyparse or yypush_parse. | -`-------------------------*/ +/*----------. +| yyparse. | +`----------*/ #ifdef YYPARSE_PARAM #if (defined __STDC__ || defined __C99__FUNC__ \ @@ -1447,7 +1469,7 @@ yybackup: /* First try to decide what to do without reference to lookahead token. */ yyn = yypact[yystate]; - if (yyn == YYPACT_NINF) + if (yypact_value_is_default (yyn)) goto yydefault; /* Not known => get a lookahead token if don't already have one. */ @@ -1478,8 +1500,8 @@ yybackup: yyn = yytable[yyn]; if (yyn <= 0) { - if (yyn == 0 || yyn == YYTABLE_NINF) - goto yyerrlab; + if (yytable_value_is_error (yyn)) + goto yyerrlab; yyn = -yyn; goto yyreduce; } @@ -1534,78 +1556,78 @@ yyreduce: { case 4: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 178 "libmemcached/csl/parser.yy" - { ;} + { } break; case 5: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 180 "libmemcached/csl/parser.yy" - { ;} + { } break; case 6: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 182 "libmemcached/csl/parser.yy" - { ;} + { } break; case 7: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 184 "libmemcached/csl/parser.yy" { context->set_end(); YYACCEPT; - ;} + } break; case 8: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 189 "libmemcached/csl/parser.yy" { context->rc= MEMCACHED_PARSE_USER_ERROR; parser_abort(context, NULL); - ;} + } break; case 9: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 194 "libmemcached/csl/parser.yy" { memcached_reset(context->memc); - ;} + } break; case 10: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 198 "libmemcached/csl/parser.yy" { yydebug= 1; - ;} + } break; case 11: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 202 "libmemcached/csl/parser.yy" { if ((context->rc= memcached_parse_configure_file(*context->memc, (yyvsp[(3) - (3)].string).c_str, (yyvsp[(3) - (3)].string).size)) != MEMCACHED_SUCCESS) { parser_abort(context, NULL); } - ;} + } break; case 12: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 213 "libmemcached/csl/parser.yy" { if (memcached_failed(context->rc= memcached_server_add_with_weight(context->memc, (yyvsp[(2) - (4)].server).c_str, (yyvsp[(3) - (4)].number), (yyvsp[(4) - (4)].number)))) @@ -1613,12 +1635,12 @@ yyreduce: parser_abort(context, NULL); } context->unset_server(); - ;} + } break; case 13: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 221 "libmemcached/csl/parser.yy" { if (memcached_failed(context->rc= memcached_server_add_with_weight(context->memc, (yyvsp[(2) - (4)].server).c_str, (yyvsp[(3) - (4)].number), (yyvsp[(4) - (4)].number)))) @@ -1626,508 +1648,524 @@ yyreduce: parser_abort(context, NULL); } context->unset_server(); - ;} + } break; case 14: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 229 "libmemcached/csl/parser.yy" { if (memcached_failed(context->rc= memcached_server_add_unix_socket_with_weight(context->memc, (yyvsp[(2) - (3)].string).c_str, (yyvsp[(3) - (3)].number)))) { parser_abort(context, NULL); } - ;} + } break; case 15: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 236 "libmemcached/csl/parser.yy" { memcached_set_configuration_file(context->memc, (yyvsp[(2) - (2)].string).c_str, (yyvsp[(2) - (2)].string).size); - ;} + } break; case 16: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 240 "libmemcached/csl/parser.yy" { context->memc->configure.initial_pool_size= (yyvsp[(2) - (2)].number); - ;} + } break; case 17: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 244 "libmemcached/csl/parser.yy" { context->memc->configure.max_pool_size= (yyvsp[(2) - (2)].number); - ;} + } break; case 19: -/* Line 1464 of yacc.c */ +/* Line 1806 of yacc.c */ #line 252 "libmemcached/csl/parser.yy" { + if (memcached_callback_get(context->memc, MEMCACHED_CALLBACK_PREFIX_KEY, NULL)) + { + parser_abort(context, "--NAMESPACE can only be called once"); + } + if ((context->rc= memcached_set_namespace(context->memc, (yyvsp[(2) - (2)].string).c_str, (yyvsp[(2) - (2)].string).size)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, memcached_last_error_message(context->memc)); } - ;} + } break; case 20: -/* Line 1464 of yacc.c */ -#line 259 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 264 "libmemcached/csl/parser.yy" { if ((context->rc= memcached_behavior_set(context->memc, MEMCACHED_BEHAVIOR_DISTRIBUTION, (yyvsp[(2) - (2)].distribution))) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, memcached_last_error_message(context->memc));; } - ;} + } break; case 21: -/* Line 1464 of yacc.c */ -#line 266 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 271 "libmemcached/csl/parser.yy" { if ((context->rc= memcached_behavior_set(context->memc, MEMCACHED_BEHAVIOR_DISTRIBUTION, (yyvsp[(2) - (4)].distribution))) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } if ((context->rc= memcached_behavior_set_distribution_hash(context->memc, (yyvsp[(4) - (4)].hash))) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } - ;} + } break; case 22: -/* Line 1464 of yacc.c */ -#line 277 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 282 "libmemcached/csl/parser.yy" { if ((context->rc= memcached_behavior_set(context->memc, MEMCACHED_BEHAVIOR_HASH, (yyvsp[(2) - (2)].hash))) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } - ;} + } break; case 23: -/* Line 1464 of yacc.c */ -#line 284 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 289 "libmemcached/csl/parser.yy" { if ((context->rc= memcached_behavior_set(context->memc, (yyvsp[(1) - (2)].behavior), (yyvsp[(2) - (2)].number))) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } - ;} + } break; case 24: -/* Line 1464 of yacc.c */ -#line 291 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 296 "libmemcached/csl/parser.yy" { if ((context->rc= memcached_behavior_set(context->memc, (yyvsp[(1) - (1)].behavior), true)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } - ;} + } break; case 25: -/* Line 1464 of yacc.c */ -#line 298 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 303 "libmemcached/csl/parser.yy" { - ;} + } break; case 26: -/* Line 1464 of yacc.c */ -#line 304 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 309 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_REMOVE_FAILED_SERVERS; - ;} + } break; case 27: -/* Line 1464 of yacc.c */ -#line 308 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 313 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_CONNECT_TIMEOUT; - ;} + } break; case 28: -/* Line 1464 of yacc.c */ -#line 312 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 317 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_IO_MSG_WATERMARK; - ;} + } break; case 29: -/* Line 1464 of yacc.c */ -#line 316 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 321 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_IO_BYTES_WATERMARK; - ;} + } break; case 30: -/* Line 1464 of yacc.c */ -#line 320 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 325 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_IO_KEY_PREFETCH; - ;} + } break; case 31: -/* Line 1464 of yacc.c */ -#line 324 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 329 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_NUMBER_OF_REPLICAS; - ;} + } break; case 32: -/* Line 1464 of yacc.c */ -#line 328 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 333 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_POLL_TIMEOUT; - ;} + } break; case 33: -/* Line 1464 of yacc.c */ -#line 332 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 337 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_RCV_TIMEOUT; - ;} + } break; case 34: -/* Line 1464 of yacc.c */ -#line 336 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 341 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_RETRY_TIMEOUT; - ;} + } break; case 35: -/* Line 1464 of yacc.c */ -#line 340 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 345 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_SND_TIMEOUT; - ;} + } break; case 36: -/* Line 1464 of yacc.c */ -#line 344 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 349 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_SOCKET_RECV_SIZE; - ;} + } break; case 37: -/* Line 1464 of yacc.c */ -#line 348 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 353 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_SOCKET_SEND_SIZE; - ;} + } break; case 38: -/* Line 1464 of yacc.c */ -#line 355 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 360 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_BINARY_PROTOCOL; - ;} + } break; case 39: -/* Line 1464 of yacc.c */ -#line 359 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 364 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_BUFFER_REQUESTS; - ;} + } break; case 40: -/* Line 1464 of yacc.c */ -#line 363 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 368 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_HASH_WITH_PREFIX_KEY; - ;} + } break; case 41: -/* Line 1464 of yacc.c */ -#line 367 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 372 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_NOREPLY; - ;} + } break; case 42: -/* Line 1464 of yacc.c */ -#line 371 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 376 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_RANDOMIZE_REPLICA_READ; - ;} + } break; case 43: -/* Line 1464 of yacc.c */ -#line 375 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 380 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_SORT_HOSTS; - ;} + } break; case 44: -/* Line 1464 of yacc.c */ -#line 379 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 384 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_SUPPORT_CAS; - ;} + } break; case 45: -/* Line 1464 of yacc.c */ -#line 383 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 388 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_TCP_NODELAY; - ;} + } break; case 46: -/* Line 1464 of yacc.c */ -#line 387 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 392 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_TCP_KEEPALIVE; - ;} + } break; case 47: -/* Line 1464 of yacc.c */ -#line 391 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 396 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_TCP_KEEPIDLE; - ;} + } break; case 48: -/* Line 1464 of yacc.c */ -#line 395 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 400 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_USE_UDP; - ;} + } break; case 49: -/* Line 1464 of yacc.c */ -#line 399 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 404 "libmemcached/csl/parser.yy" { (yyval.behavior)= MEMCACHED_BEHAVIOR_VERIFY_KEY; - ;} + } break; case 50: -/* Line 1464 of yacc.c */ -#line 405 "libmemcached/csl/parser.yy" - { (yyval.number)= MEMCACHED_DEFAULT_PORT;;} +/* Line 1806 of yacc.c */ +#line 410 "libmemcached/csl/parser.yy" + { (yyval.number)= MEMCACHED_DEFAULT_PORT;} break; case 51: -/* Line 1464 of yacc.c */ -#line 407 "libmemcached/csl/parser.yy" - { ;} +/* Line 1806 of yacc.c */ +#line 412 "libmemcached/csl/parser.yy" + { } break; case 52: -/* Line 1464 of yacc.c */ -#line 411 "libmemcached/csl/parser.yy" - { (yyval.number)= 1; ;} +/* Line 1806 of yacc.c */ +#line 416 "libmemcached/csl/parser.yy" + { (yyval.number)= 1; } break; case 53: -/* Line 1464 of yacc.c */ -#line 413 "libmemcached/csl/parser.yy" - { ;} +/* Line 1806 of yacc.c */ +#line 418 "libmemcached/csl/parser.yy" + { } break; case 54: -/* Line 1464 of yacc.c */ -#line 418 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 423 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_MD5; - ;} + } break; case 55: -/* Line 1464 of yacc.c */ -#line 422 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 427 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_CRC; - ;} + } break; case 56: -/* Line 1464 of yacc.c */ -#line 426 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 431 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_FNV1_64; - ;} + } break; case 57: -/* Line 1464 of yacc.c */ -#line 430 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 435 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_FNV1A_64; - ;} + } break; case 58: -/* Line 1464 of yacc.c */ -#line 434 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 439 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_FNV1_32; - ;} + } break; case 59: -/* Line 1464 of yacc.c */ -#line 438 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 443 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_FNV1A_32; - ;} + } break; case 60: -/* Line 1464 of yacc.c */ -#line 442 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 447 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_HSIEH; - ;} + } break; case 61: -/* Line 1464 of yacc.c */ -#line 446 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 451 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_MURMUR; - ;} + } break; case 62: -/* Line 1464 of yacc.c */ -#line 450 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 455 "libmemcached/csl/parser.yy" { (yyval.hash)= MEMCACHED_HASH_JENKINS; - ;} + } break; case 63: -/* Line 1464 of yacc.c */ -#line 457 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 462 "libmemcached/csl/parser.yy" { (yyval.string)= (yyvsp[(1) - (1)].string); - ;} + } break; case 64: -/* Line 1464 of yacc.c */ -#line 461 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 466 "libmemcached/csl/parser.yy" { (yyval.string)= (yyvsp[(1) - (1)].string); - ;} + } break; case 65: -/* Line 1464 of yacc.c */ -#line 468 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 473 "libmemcached/csl/parser.yy" { (yyval.distribution)= MEMCACHED_DISTRIBUTION_CONSISTENT; - ;} + } break; case 66: -/* Line 1464 of yacc.c */ -#line 472 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 477 "libmemcached/csl/parser.yy" { (yyval.distribution)= MEMCACHED_DISTRIBUTION_MODULA; - ;} + } break; case 67: -/* Line 1464 of yacc.c */ -#line 476 "libmemcached/csl/parser.yy" +/* Line 1806 of yacc.c */ +#line 481 "libmemcached/csl/parser.yy" { (yyval.distribution)= MEMCACHED_DISTRIBUTION_RANDOM; - ;} + } break; -/* Line 1464 of yacc.c */ -#line 2129 "libmemcached/csl/parser.cc" +/* Line 1806 of yacc.c */ +#line 2156 "libmemcached/csl/parser.cc" default: break; } + /* User semantic actions sometimes alter yychar, and that requires + that yytoken be updated with the new translation. We take the + approach of translating immediately before every use of yytoken. + One alternative is translating here after every semantic action, + but that translation would be missed if the semantic action invokes + YYABORT, YYACCEPT, or YYERROR immediately after altering yychar or + if it invokes YYBACKUP. In the case of YYABORT or YYACCEPT, an + incorrect destructor might then be invoked immediately. In the + case of YYERROR or YYBACKUP, subsequent parser actions might lead + to an incorrect destructor call or verbose syntax error message + before the lookahead is translated. */ YY_SYMBOL_PRINT ("-> $$ =", yyr1[yyn], &yyval, &yyloc); YYPOPSTACK (yylen); @@ -2155,6 +2193,10 @@ yyreduce: | yyerrlab -- here on detecting error | `------------------------------------*/ yyerrlab: + /* Make sure we have latest lookahead translation. See comments at + user semantic actions for why this is necessary. */ + yytoken = yychar == YYEMPTY ? YYEMPTY : YYTRANSLATE (yychar); + /* If not already recovering from an error, report this error. */ if (!yyerrstatus) { @@ -2162,37 +2204,36 @@ yyerrlab: #if ! YYERROR_VERBOSE yyerror (context, scanner, YY_("syntax error")); #else +# define YYSYNTAX_ERROR yysyntax_error (&yymsg_alloc, &yymsg, \ + yyssp, yytoken) { - YYSIZE_T yysize = yysyntax_error (0, yystate, yychar); - if (yymsg_alloc < yysize && yymsg_alloc < YYSTACK_ALLOC_MAXIMUM) - { - YYSIZE_T yyalloc = 2 * yysize; - if (! (yysize <= yyalloc && yyalloc <= YYSTACK_ALLOC_MAXIMUM)) - yyalloc = YYSTACK_ALLOC_MAXIMUM; - if (yymsg != yymsgbuf) - YYSTACK_FREE (yymsg); - yymsg = (char *) YYSTACK_ALLOC (yyalloc); - if (yymsg) - yymsg_alloc = yyalloc; - else - { - yymsg = yymsgbuf; - yymsg_alloc = sizeof yymsgbuf; - } - } - - if (0 < yysize && yysize <= yymsg_alloc) - { - (void) yysyntax_error (yymsg, yystate, yychar); - yyerror (context, scanner, yymsg); - } - else - { - yyerror (context, scanner, YY_("syntax error")); - if (yysize != 0) - goto yyexhaustedlab; - } + char const *yymsgp = YY_("syntax error"); + int yysyntax_error_status; + yysyntax_error_status = YYSYNTAX_ERROR; + if (yysyntax_error_status == 0) + yymsgp = yymsg; + else if (yysyntax_error_status == 1) + { + if (yymsg != yymsgbuf) + YYSTACK_FREE (yymsg); + yymsg = (char *) YYSTACK_ALLOC (yymsg_alloc); + if (!yymsg) + { + yymsg = yymsgbuf; + yymsg_alloc = sizeof yymsgbuf; + yysyntax_error_status = 2; + } + else + { + yysyntax_error_status = YYSYNTAX_ERROR; + yymsgp = yymsg; + } + } + yyerror (context, scanner, yymsgp); + if (yysyntax_error_status == 2) + goto yyexhaustedlab; } +# undef YYSYNTAX_ERROR #endif } @@ -2251,7 +2292,7 @@ yyerrlab1: for (;;) { yyn = yypact[yystate]; - if (yyn != YYPACT_NINF) + if (!yypact_value_is_default (yyn)) { yyn += YYTERROR; if (0 <= yyn && yyn <= YYLAST && yycheck[yyn] == YYTERROR) @@ -2310,8 +2351,13 @@ yyexhaustedlab: yyreturn: if (yychar != YYEMPTY) - yydestruct ("Cleanup: discarding lookahead", - yytoken, &yylval, context, scanner); + { + /* Make sure we have latest lookahead translation. See comments at + user semantic actions for why this is necessary. */ + yytoken = YYTRANSLATE (yychar); + yydestruct ("Cleanup: discarding lookahead", + yytoken, &yylval, context, scanner); + } /* Do not reclaim the symbols of the rule which action triggered this YYABORT or YYACCEPT. */ YYPOPSTACK (yylen); @@ -2336,8 +2382,8 @@ yyreturn: -/* Line 1684 of yacc.c */ -#line 481 "libmemcached/csl/parser.yy" +/* Line 2067 of yacc.c */ +#line 486 "libmemcached/csl/parser.yy" void Context::start() diff --git a/libmemcached/csl/parser.h b/libmemcached/csl/parser.h index 16657d24..4112d748 100644 --- a/libmemcached/csl/parser.h +++ b/libmemcached/csl/parser.h @@ -1,9 +1,8 @@ -/* A Bison parser, made by GNU Bison 2.4.3. */ +/* A Bison parser, made by GNU Bison 2.5. */ -/* Skeleton interface for Bison's Yacc-like parsers in C +/* Bison interface for Yacc-like parsers in C - Copyright (C) 1984, 1989, 1990, 2000, 2001, 2002, 2003, 2004, 2005, 2006, - 2009, 2010 Free Software Foundation, Inc. + Copyright (C) 1984, 1989-1990, 2000-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 diff --git a/libmemcached/csl/parser.yy b/libmemcached/csl/parser.yy index 08441987..332d323e 100644 --- a/libmemcached/csl/parser.yy +++ b/libmemcached/csl/parser.yy @@ -250,48 +250,53 @@ expression: behaviors: NAMESPACE string { + if (memcached_callback_get(context->memc, MEMCACHED_CALLBACK_PREFIX_KEY, NULL)) + { + parser_abort(context, "--NAMESPACE can only be called once"); + } + if ((context->rc= memcached_set_namespace(context->memc, $2.c_str, $2.size)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, memcached_last_error_message(context->memc)); } } | DISTRIBUTION distribution { if ((context->rc= memcached_behavior_set(context->memc, MEMCACHED_BEHAVIOR_DISTRIBUTION, $2)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, memcached_last_error_message(context->memc));; } } | DISTRIBUTION distribution ',' hash { if ((context->rc= memcached_behavior_set(context->memc, MEMCACHED_BEHAVIOR_DISTRIBUTION, $2)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } if ((context->rc= memcached_behavior_set_distribution_hash(context->memc, $4)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } } | HASH hash { if ((context->rc= memcached_behavior_set(context->memc, MEMCACHED_BEHAVIOR_HASH, $2)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } } | behavior_number NUMBER { if ((context->rc= memcached_behavior_set(context->memc, $1, $2)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } } | behavior_boolean { if ((context->rc= memcached_behavior_set(context->memc, $1, true)) != MEMCACHED_SUCCESS) { - parser_abort(context, NULL);; + parser_abort(context, NULL); } } | USER_DATA diff --git a/libmemcached/include.am b/libmemcached/include.am index e3f5fa3f..e09b8a67 100644 --- a/libmemcached/include.am +++ b/libmemcached/include.am @@ -97,6 +97,7 @@ libmemcached/options.cc: libmemcached/csl/parser.h libmemcached_libmemcached_la_DEPENDENCIES= libmemcached_libmemcached_la_LIBADD= +libmemcached_libmemcached_la_LDFLAGS= if TARGET_LINUX libmemcached_libmemcached_la_LIBADD+= -lm endif diff --git a/libmemcached/strerror.cc b/libmemcached/strerror.cc index ebb90056..b70a60b9 100644 --- a/libmemcached/strerror.cc +++ b/libmemcached/strerror.cc @@ -185,6 +185,9 @@ const char *memcached_strerror(memcached_st *, memcached_return_t rc) case MEMCACHED_SERVER_TEMPORARILY_DISABLED: return "SERVER HAS FAILED AND IS DISABLED UNTIL TIMED RETRY"; + case MEMCACHED_SERVER_MEMORY_ALLOCATION_FAILURE: + return "SERVER FAILED TO ALLOCATE OBJECT"; + default: case MEMCACHED_MAXIMUM_RETURN: return "INVALID memcached_return_t"; diff --git a/libmemcachedutil/ping.cc b/libmemcachedutil/ping.cc index 82d891a9..28d11081 100644 --- a/libmemcachedutil/ping.cc +++ b/libmemcachedutil/ping.cc @@ -42,11 +42,13 @@ bool libmemcached_util_ping(const char *hostname, in_port_t port, memcached_return_t *ret) { memcached_return_t unused; - if (not ret) + if (ret == NULL) + { ret= &unused; + } memcached_st *memc_ptr= memcached_create(NULL); - if (not memc_ptr) + if (memc_ptr == NULL) { *ret= MEMCACHED_MEMORY_ALLOCATION_FAILURE; return false; diff --git a/libtest/include.am b/libtest/include.am index a52c573a..a668d690 100644 --- a/libtest/include.am +++ b/libtest/include.am @@ -35,6 +35,9 @@ export GDB_COMMAND valgrind: @echo make check TESTS_ENVIRONMENT="\"$(VALGRIND_COMMAND)\"" +gdb: + @echo make check TESTS_ENVIRONMENT="\"$(GDB_COMMAND)\"" + helgrind: @echo make check TESTS_ENVIRONMENT="\"$(HELGRIND_COMMAND)\"" diff --git a/tests/mem_functions.cc b/tests/mem_functions.cc index 03dc591d..a625036f 100644 --- a/tests/mem_functions.cc +++ b/tests/mem_functions.cc @@ -485,7 +485,7 @@ static test_return_t memcached_return_t_TEST(memcached_st *memc) 4159057246U, 3425930182U, 2593724503U, 1868899624U, 1769812374U, 2302537950U, 1110330676U, 3365377466U, 1336171666U, 3021258493U, 2334992265U, 3861994737U, - 3365377466U }; + 3582734124, 3365377466U }; // You have updated the memcache_error messages but not updated docs/tests. for (int rc= int(MEMCACHED_SUCCESS); rc < int(MEMCACHED_MAXIMUM_RETURN); ++rc) @@ -501,7 +501,7 @@ static test_return_t memcached_return_t_TEST(memcached_st *memc) } test_compare(values[rc], hash_val); } - test_compare(48, int(MEMCACHED_MAXIMUM_RETURN)); + test_compare(49, int(MEMCACHED_MAXIMUM_RETURN)); return TEST_SUCCESS; } @@ -1854,11 +1854,10 @@ static test_return_t block_add_regression(memcached_st *memc) /* First add all of the items.. */ for (size_t x= 0; x < REGRESSION_BINARY_VS_BLOCK_COUNT; ++x) { - memcached_return_t rc; char blob[1024] = {0}; - rc= memcached_add_by_key(memc, "bob", 3, global_pairs[x].key, global_pairs[x].key_length, blob, sizeof(blob), 0, 0); - test_true(rc == MEMCACHED_SUCCESS || rc == MEMCACHED_BUFFERED); + memcached_return_t rc= memcached_add_by_key(memc, "bob", 3, global_pairs[x].key, global_pairs[x].key_length, blob, sizeof(blob), 0, 0); + test_true_got(rc == MEMCACHED_SUCCESS or rc == MEMCACHED_SERVER_MEMORY_ALLOCATION_FAILURE, memcached_strerror(NULL, rc)); } return TEST_SUCCESS; @@ -1866,9 +1865,9 @@ static test_return_t block_add_regression(memcached_st *memc) static test_return_t binary_add_regression(memcached_st *memc) { - memcached_behavior_set(memc, MEMCACHED_BEHAVIOR_BINARY_PROTOCOL, 1); + test_skip(MEMCACHED_SUCCESS, memcached_behavior_set(memc, MEMCACHED_BEHAVIOR_BINARY_PROTOCOL, true)); test_return_t rc= block_add_regression(memc); - memcached_behavior_set(memc, MEMCACHED_BEHAVIOR_BINARY_PROTOCOL, 0); + return rc; } @@ -2240,10 +2239,11 @@ static test_return_t user_supplied_bug3(memcached_st *memc) getter = memcached_behavior_get(memc, MEMCACHED_BEHAVIOR_SOCKET_RECV_SIZE); #endif - size_t key_lengths[HALDENBRAND_KEY_COUNT]; - char **keys= static_cast(calloc(HALDENBRAND_KEY_COUNT, sizeof(char *))); + std::vector key_lengths; + key_lengths.resize(HALDENBRAND_KEY_COUNT); + char **keys= static_cast(calloc(key_lengths.size(), sizeof(char *))); test_true(keys); - for (uint32_t x= 0; x < HALDENBRAND_KEY_COUNT; x++) + for (uint32_t x= 0; x < key_lengths.size(); x++) { char key[MEMCACHED_MAXIMUM_INTEGER_DISPLAY_LENGTH +1]; int key_length= snprintf(key, sizeof(key), "%u", x); @@ -2255,7 +2255,7 @@ static test_return_t user_supplied_bug3(memcached_st *memc) } test_compare(MEMCACHED_SUCCESS, - memcached_mget(memc, (const char **)keys, key_lengths, HALDENBRAND_KEY_COUNT)); + memcached_mget(memc, (const char **)keys, &key_lengths[0], key_lengths.size())); unsigned int keys_returned; test_compare(TEST_SUCCESS, fetch_all_results(memc, keys_returned, MEMCACHED_SUCCESS)); @@ -2498,11 +2498,11 @@ static test_return_t user_supplied_bug7(memcached_st *memc) memcached_flush(memc, 0); flags= 245; - memcached_return_t rc= memcached_set(memc, keys, key_length, - insert_data, VALUE_SIZE_BUG5, - (time_t)0, flags); - test_compare(MEMCACHED_SUCCESS, rc); + test_compare(MEMCACHED_SUCCESS, memcached_set(memc, keys, key_length, + insert_data, VALUE_SIZE_BUG5, + (time_t)0, flags)); + memcached_return_t rc; flags= 0; value= memcached_get(memc, keys, key_length, &value_length, &flags, &rc); @@ -2651,12 +2651,12 @@ static test_return_t user_supplied_bug12(memcached_st *memc) value= memcached_get(memc, "autoincrement", strlen("autoincrement"), &value_length, &flags, &rc); - test_true(value == NULL); + test_null(value); test_compare(MEMCACHED_NOTFOUND, rc); rc= memcached_increment(memc, "autoincrement", strlen("autoincrement"), 1, &number_value); - test_true(value == NULL); + test_null(value); /* The binary protocol will set the key if it doesn't exist */ if (memcached_behavior_get(memc, MEMCACHED_BEHAVIOR_BINARY_PROTOCOL) == 1) { @@ -2667,18 +2667,16 @@ static test_return_t user_supplied_bug12(memcached_st *memc) test_compare(MEMCACHED_NOTFOUND, rc); } - rc= memcached_set(memc, "autoincrement", strlen("autoincrement"), "1", 1, 0, 0); + test_compare(MEMCACHED_SUCCESS, + memcached_set(memc, "autoincrement", strlen("autoincrement"), "1", 1, 0, 0)); - value= memcached_get(memc, "autoincrement", strlen("autoincrement"), - &value_length, &flags, &rc); + value= memcached_get(memc, "autoincrement", strlen("autoincrement"), &value_length, &flags, &rc); test_true(value); - test_compare(MEMCACHED_SUCCESS, rc); free(value); - rc= memcached_increment(memc, "autoincrement", strlen("autoincrement"), - 1, &number_value); - test_true(number_value == 2); - test_compare(MEMCACHED_SUCCESS, rc); + test_compare(MEMCACHED_SUCCESS, + memcached_increment(memc, "autoincrement", strlen("autoincrement"), 1, &number_value)); + test_compare(2UL, number_value); return TEST_SUCCESS; } @@ -2923,11 +2921,11 @@ static test_return_t _user_supplied_bug21(memcached_st* memc, size_t key_count) /* empty the cache to ensure misses (hence non-responses) */ test_compare(MEMCACHED_SUCCESS, memcached_flush(memc_clone, 0)); - size_t* key_lengths= new (std::nothrow) size_t[key_count]; - test_true(key_lengths); - char **keys= static_cast(calloc(key_count, sizeof(char *))); + std::vector key_lengths; + key_lengths.resize(key_count); + char **keys= static_cast(calloc(key_lengths.size(), sizeof(char *))); test_true(keys); - for (unsigned int x= 0; x < key_count; x++) + for (unsigned int x= 0; x < key_lengths.size(); x++) { char buffer[30]; @@ -2941,7 +2939,7 @@ static test_return_t _user_supplied_bug21(memcached_st* memc, size_t key_count) alarm(5); test_compare_got(MEMCACHED_SUCCESS, - memcached_mget(memc_clone, (const char **)keys, key_lengths, key_count), memcached_last_error_message(memc_clone)); + memcached_mget(memc_clone, (const char **)keys, &key_lengths[0], key_count), memcached_last_error_message(memc_clone)); alarm(0); signal(SIGALRM, oldalarm); @@ -2968,7 +2966,6 @@ static test_return_t _user_supplied_bug21(memcached_st* memc, size_t key_count) free(keys[x]); } free(keys); - delete [] key_lengths; memcached_free(memc_clone); @@ -5642,7 +5639,9 @@ test_st tests[] ={ {"analyzer", true, (test_callback_fn*)analyzer_test}, {"memcached_pool_st", true, (test_callback_fn*)connection_pool_test }, {"memcached_pool_st #2", true, (test_callback_fn*)connection_pool2_test }, +#if 0 {"memcached_pool_st #3", true, (test_callback_fn*)connection_pool3_test }, +#endif {"memcached_pool_test", true, (test_callback_fn*)memcached_pool_test }, {"test_get_last_disconnect", true, (test_callback_fn*)test_get_last_disconnect}, {"verbosity", true, (test_callback_fn*)test_verbosity}, diff --git a/tests/mem_udp.cc b/tests/mem_udp.cc index e33610da..fc469bae 100644 --- a/tests/mem_udp.cc +++ b/tests/mem_udp.cc @@ -173,6 +173,8 @@ static test_return_t udp_set_test(memcached_st *memc) { unsigned int num_iters= 1025; //request id rolls over at 1024 + test_true(memc); + for (size_t x= 0; x < num_iters;x++) { Expected expected_ids; @@ -183,7 +185,7 @@ static test_return_t udp_set_test(memcached_st *memc) memcached_return_t rc= memcached_set(memc, test_literal_param("foo"), test_literal_param("when we sanitize"), - (time_t)0, (uint32_t)0); + time_t(0), uint32_t(0)); test_true(rc == MEMCACHED_SUCCESS or rc == MEMCACHED_BUFFERED); /** NB, the check below assumes that if new write_ptr is less than * the original write_ptr that we have flushed. For large payloads, this @@ -212,12 +214,14 @@ static test_return_t udp_set_test(memcached_st *memc) static test_return_t udp_buffered_set_test(memcached_st *memc) { + test_true(memc); test_compare(MEMCACHED_INVALID_ARGUMENTS, memcached_behavior_set(memc, MEMCACHED_BEHAVIOR_BUFFER_REQUESTS, true)); return TEST_SUCCESS; } static test_return_t udp_set_too_big_test(memcached_st *memc) { + test_true(memc); char value[MAX_UDP_DATAGRAM_LENGTH]; Expected expected_ids; get_udp_request_ids(memc, expected_ids); @@ -232,6 +236,8 @@ static test_return_t udp_set_too_big_test(memcached_st *memc) static test_return_t udp_delete_test(memcached_st *memc) { + test_true(memc); + //request id rolls over at 1024 for (size_t x= 0; x < 1025; x++) { diff --git a/tests/parser.cc b/tests/parser.cc index bfcb0816..d514c45f 100644 --- a/tests/parser.cc +++ b/tests/parser.cc @@ -41,7 +41,6 @@ using namespace libtest; #include -#include #include #include #include @@ -456,6 +455,7 @@ test_return_t random_statement_build_test(memcached_st*) for (scanner_variable_t *ptr= hash_strings; ptr->type != NIL; ptr++) option_list.push_back(&ptr->option); + bool seen_namespace= false; for (uint32_t x= 0; x < RANDOM_STRINGS; x++) { std::string random_options; @@ -463,38 +463,36 @@ test_return_t random_statement_build_test(memcached_st*) uint32_t number_of= random() % option_list.size(); for (uint32_t options= 0; options < number_of; options++) { - random_options+= option_list[random() % option_list.size()]->c_str; + std::string random_string= option_list[random() % option_list.size()]->c_str; + bool is_namespace= memcmp(random_string.c_str(), test_literal_param("--NAMESPACE")) == 0; + + if (is_namespace and seen_namespace) + { + continue; + } + + if (is_namespace) + { + seen_namespace= true; + } + + random_options+= random_string; random_options+= " "; } - memcached_st *memc_ptr= memcached(random_options.c_str(), random_options.size() -1); - if (not memc_ptr) + if (random_options.size() <= 1) { - switch (errno) - { - case EINVAL: -#if 0 // Testing framework is not smart enough for this just yet. - { - // We will try to find the specific error - char buffer[2048]; - memcached_return_t rc= libmemcached_check_configuration(random_options.c_str(), random_options.size(), buffer, sizeof(buffer)); - test_true_got(rc != MEMCACHED_SUCCESS, "memcached_create_with_options() failed whiled libmemcached_check_configuration() was successful"); - std::cerr << "Error occured on " << random_options.c_str() << " : " << buffer << std::endl; - return TEST_FAILURE; - } -#endif - break; - case ENOMEM: - std::cerr << "Failed to allocate memory for memcached_create_with_options()" << std::endl; - memcached_free(memc_ptr); - return TEST_FAILURE; - default: - std::cerr << "Unknown error from memcached_create_with_options?!!" << std::endl; - memcached_free(memc_ptr); - return TEST_FAILURE; - } + continue; + } + + char buffer[BUFSIZ]; + memcached_return_t rc= libmemcached_check_configuration(random_options.c_str(), random_options.size() -1, buffer, sizeof(buffer)); + + if (memcached_failed(rc)) + { + Error << "libmemcached_check_configuration(" << random_options << ") : " << buffer; + return TEST_FAILURE; } - memcached_free(memc_ptr); } return TEST_SUCCESS; @@ -558,7 +556,7 @@ static memcached_return_t dump_socket_information(const memcached_st *, if (strcmp(memcached_server_name(instance), check->socket)) { - std::cerr << std::endl << __FILE__ << ":" << __LINE__ << " " << memcached_server_name(instance) << " != " << check->socket << std::endl; + Error << memcached_server_name(instance) << " != " << check->socket; return MEMCACHED_FAILURE; }