From 387b37c1acab0791f19cfced1e345543855d764e Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Tue, 25 Aug 2015 21:17:47 +0000 Subject: [PATCH] compiler: Accept numeric literals with leading zeroes. When a numeric literal with leading zeroes was seen in the parser, it would only be accepted if it were a valid hex or octal literal. Any invalid numeric literal would be split up into multiple tokens: the valid hex/octal literal followed by the rest of the characters. Instead, when scanning a numeric literal with leading zeroes, always accept the number and give an appropriate error if the accepted number does not fit in the expected base. Fixes golang/go#11532, golang/go#11533. Reviewed-on: https://go-review.googlesource.com/13791 From-SVN: r227193 --- gcc/go/gofrontend/MERGE | 2 +- gcc/go/gofrontend/lex.cc | 10 ++++++++-- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index 86d163b1f52..c30efd919ec 100644 --- a/gcc/go/gofrontend/MERGE +++ b/gcc/go/gofrontend/MERGE @@ -1,4 +1,4 @@ -14ca4b6130b9a7132d132f418e9ea283b3a52c08 +f97d579fa8431af5cfde9b0a48604caabfd09377 The first line of this file holds the git revision number of the last merge done from the gofrontend repository. diff --git a/gcc/go/gofrontend/lex.cc b/gcc/go/gofrontend/lex.cc index 8de94c5114b..632a1c9e353 100644 --- a/gcc/go/gofrontend/lex.cc +++ b/gcc/go/gofrontend/lex.cc @@ -1047,7 +1047,7 @@ Lex::gather_number() pnum = p; while (p < pend) { - if (*p < '0' || *p > '7') + if (*p < '0' || *p > '9') break; ++p; } @@ -1060,7 +1060,13 @@ Lex::gather_number() std::string s(pnum, p - pnum); mpz_t val; int r = mpz_init_set_str(val, s.c_str(), base); - go_assert(r == 0); + if (r != 0) + { + if (base == 8) + error_at(this->location(), "invalid octal literal"); + else + error_at(this->location(), "invalid hex literal"); + } if (neg) mpz_neg(val, val); -- 2.30.2