From e40c908b95aff31a5527abbd0690cc61f5f50bd4 Mon Sep 17 00:00:00 2001
From: Adeniyi Mayokun <adeniyimayokun17@gmail.com>
Date: Sat, 24 Mar 2018 11:07:17 +0100
Subject: [PATCH 2/2] Fixed wrc gives parse error if resource nameID has quoted
��& Updated parse.l file
��
��Signed-off-by: Adeniyi Mayokun <adeniyimayokun17@gmail.com>

---
��tools/wrc/parser.l | 102 +++++++++++++++++++++++++++++++++++++++++++++++++++--
��1 file changed, 99 insertions(+), 3 deletions(-)

diff --git a/tools/wrc/parser.l b/tools/wrc/parser.l
index 32a231b5a2..dfcefc069e 100644
--- a/tools/wrc/parser.l
+++ b/tools/wrc/parser.l
@@ -66,7 +66,10 @@
�� *�� - Rebuild string processing so that it may contain
�� *�� escaped '\0'.
�� */
-
+/* Exclusive double-quoted nameID handling */
+%x tkid_dbl
+/* Exclusive single-quoted nameID handling */
+%x tkid_sgl
��/* Exclusive string handling */
��%x tkstr
��/* Exclusive unicode string handling */
@@ -122,6 +125,7 @@ ws�� [ \f\t\r]
��#define YY_USER_ACTION�� char_number+=yyleng; wanted_id = want_id; want_id = 0;
��
��#define YY_USER_INIT current_codepage = -1;
+id_initial = 1;
��
��static void addcchar(char c);
��static void addwchar(WCHAR s);
@@ -138,6 +142,13 @@ static int wbufalloc = 0;
��
��static int current_codepage = -1;�� /* use language default */
��
+/* Used for differentiating between single- and double-quoted nameID's */
+static int id_dblquote = 0;
+/* Used for catching first (quoted) nameID before wanted_id is set.
+ * This variable is reset to 0 once the first ID (quoted or not), string or
+ * data has been encountered */
+static int id_initial = 1;
+
��/*
�� * This one is a bit tricky.
�� * We set 'want_id' in the parser to get the first
@@ -402,8 +413,46 @@ static unsigned long xstrtoul(const char *nptr, char **endptr, int base)
��0[oO][0-7]+[lL]?�� { parser_lval.num = xstrtoul(yytext+2, 0, 8);
�� return (yytext[yyleng-1] == 'L' || yytext[yyleng-1] == 'l') ? tLNUMBER : tNUMBER; }
��
+<tkid_dbl>{A-Za-z_0-9./\\\']+\"
+|
+<tkid_sgl>[A-Za-z_0-9./\\\"]+\'
+{
+
+char *tmp;
+
+size_t len;
+
+/* Plus 1 for the initial double quote that we missed and plus 1 for a trailing NUL */
+
+len = strlen(yytext) +2;
+
+tmp = xmalloc(len);
+
+/* Put the quote character that was missed */
+
+tmp[0] = id_dblquote ? '\"' : '\'';
+
+id_dlquote = 0;
+
+strcpy(tmp + 1, yytext);
+
+/* Make sure we have a NUL at the end */
+
+tmp[len-1] = '\0';
+
+parser_lval.str = make_string(tmp);
+
+free(tmp);
+
+yy_pop_state();
+
+return tIDENT;
+}
+
+
��[A-Za-z_0-9./\\]+�� {
�� struct keyword *tok = iskeyword(yytext);
+�� id_initial = 0;
��
�� if(tok)
�� {
@@ -427,6 +476,9 @@ static unsigned long xstrtoul(const char *nptr, char **endptr, int base)
�� */
��L\"�� {
�� yy_push_state(tklstr);
+
+�� id_initial = 0;
+��
�� wbufidx = 0;
�� if(!win32)
�� parser_warning("16bit resource contains unicode strings\n");
@@ -483,7 +535,29 @@ L\"�� {
�� /*
�� * Normal string scanning
�� */
-\"�� yy_push_state(tkstr); cbufidx = 0;
+\" {
+
+
+if (wanted_id || id_initial)
+
+{
+
+yy_push_state(tkid_dbl);
+
+id_dblquote = 1;
+
+id_initial = 0;
+
+}
+
+else
+
+{
+
+�� yy_push_state(tkstr); cbufidx = 0;
+
+}
+�� }
��<tkstr>\"{ws}+�� |
��<tkstr>\"�� {
�� yy_pop_state();
@@ -528,7 +602,29 @@ L\"�� {
�� /*
�� * Raw data scanning
�� */
-\'�� yy_push_state(tkrcd); cbufidx = 0;
+\' {
+
+if (wanted_id || id_initial)
+
+{
+
+yy_push_state(tkid_sgl);
+
+id_initial = 0;
+
+id_dblquote = 0;
+
+}
+
+else
+
+{
+
+
+�� yy_push_state(tkrcd); cbufidx = 0;
+
+}
+�� }
��<tkrcd>\'�� {
�� yy_pop_state();
�� parser_lval.raw = new_raw_data();
--
2.14.1