maride/php-parser
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

#44: fix crash on an empty list item

Browse Source
This commit is contained in:
z7zmey
2018-06-26 11:57:17 +03:00
parent 26117164d8
commit 10c63a30ae
7 changed files with 137 additions and 82 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
php5/parser.go
View File

@@ -108,6 +108,9 @@ func (l *Parser) GetPositions() parser.Positions {
// helpers
func lastNode(nn []node.Node) node.Node {
if len(nn) == 0 {
return nil
}
return nn[len(nn)-1]
}

99
php5/php5.go
View File

@@ -346,7 +346,8 @@ const yyEofCode = 1
const yyErrCode = 2
const yyInitialStackSize = 16
//line php5/php5.y:6782
//line php5/php5.y:6788
type simpleIndirectReference struct {
all []*expr.Variable
last *expr.Variable
@@ -8737,16 +8738,22 @@ yydefault:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6187
{
if len(yyDollar[1].list) == 0 {
yyDollar[1].list = []node.Node{nil}
}
yyVAL.list = append(yyDollar[1].list, yyDollar[3].node)
// save comments
yylex.(*Parser).comments.AddFromToken(lastNode(yyDollar[1].list), yyDollar[2].token, comment.CommaToken)
if lastNode(yyDollar[1].list) != nil {
yylex.(*Parser).comments.AddFromToken(lastNode(yyDollar[1].list), yyDollar[2].token, comment.CommaToken)
}
yylex.(*Parser).returnTokenToPool(yyDollar, &yyVAL)
}
case 477:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6196
//line php5/php5.y:6202
{
if yyDollar[1].node == nil {
yyVAL.list = []node.Node{}
@@ -8758,7 +8765,7 @@ yydefault:
}
case 478:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6210
//line php5/php5.y:6216
{
yyVAL.node = expr.NewArrayItem(nil, yyDollar[1].node)
@@ -8769,7 +8776,7 @@ yydefault:
}
case 479:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6219
//line php5/php5.y:6225
{
item := expr.NewList(yyDollar[3].list)
yyVAL.node = expr.NewArrayItem(nil, item)
@@ -8787,7 +8794,7 @@ yydefault:
}
case 480:
yyDollar = yyS[yypt-0 : yypt+1]
//line php5/php5.y:6235
//line php5/php5.y:6241
{
yyVAL.node = nil
@@ -8795,7 +8802,7 @@ yydefault:
}
case 481:
yyDollar = yyS[yypt-0 : yypt+1]
//line php5/php5.y:6245
//line php5/php5.y:6251
{
yyVAL.list = []node.Node{}
@@ -8803,7 +8810,7 @@ yydefault:
}
case 482:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6251
//line php5/php5.y:6257
{
yyVAL.list = yyDollar[1].list
@@ -8816,7 +8823,7 @@ yydefault:
}
case 483:
yyDollar = yyS[yypt-5 : yypt+1]
//line php5/php5.y:6265
//line php5/php5.y:6271
{
arrayItem := expr.NewArrayItem(yyDollar[3].node, yyDollar[5].node)
yyVAL.list = append(yyDollar[1].list, arrayItem)
@@ -8832,7 +8839,7 @@ yydefault:
}
case 484:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6279
//line php5/php5.y:6285
{
arrayItem := expr.NewArrayItem(nil, yyDollar[3].node)
yyVAL.list = append(yyDollar[1].list, arrayItem)
@@ -8847,7 +8854,7 @@ yydefault:
}
case 485:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6292
//line php5/php5.y:6298
{
arrayItem := expr.NewArrayItem(yyDollar[1].node, yyDollar[3].node)
yyVAL.list = []node.Node{arrayItem}
@@ -8862,7 +8869,7 @@ yydefault:
}
case 486:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6305
//line php5/php5.y:6311
{
arrayItem := expr.NewArrayItem(nil, yyDollar[1].node)
yyVAL.list = []node.Node{arrayItem}
@@ -8874,7 +8881,7 @@ yydefault:
}
case 487:
yyDollar = yyS[yypt-6 : yypt+1]
//line php5/php5.y:6315
//line php5/php5.y:6321
{
reference := expr.NewReference(yyDollar[6].node)
arrayItem := expr.NewArrayItem(yyDollar[3].node, reference)
@@ -8893,7 +8900,7 @@ yydefault:
}
case 488:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6332
//line php5/php5.y:6338
{
reference := expr.NewReference(yyDollar[4].node)
arrayItem := expr.NewArrayItem(nil, reference)
@@ -8911,7 +8918,7 @@ yydefault:
}
case 489:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6348
//line php5/php5.y:6354
{
reference := expr.NewReference(yyDollar[4].node)
arrayItem := expr.NewArrayItem(yyDollar[1].node, reference)
@@ -8929,7 +8936,7 @@ yydefault:
}
case 490:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6364
//line php5/php5.y:6370
{
reference := expr.NewReference(yyDollar[2].node)
arrayItem := expr.NewArrayItem(nil, reference)
@@ -8946,7 +8953,7 @@ yydefault:
}
case 491:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6382
//line php5/php5.y:6388
{
yyVAL.list = append(yyDollar[1].list, yyDollar[2].node)
@@ -8954,7 +8961,7 @@ yydefault:
}
case 492:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6388
//line php5/php5.y:6394
{
encapsed := scalar.NewEncapsedStringPart(yyDollar[2].token.Value)
yyVAL.list = append(yyDollar[1].list, encapsed)
@@ -8969,7 +8976,7 @@ yydefault:
}
case 493:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6401
//line php5/php5.y:6407
{
yyVAL.list = []node.Node{yyDollar[1].node}
@@ -8977,7 +8984,7 @@ yydefault:
}
case 494:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6407
//line php5/php5.y:6413
{
encapsed := scalar.NewEncapsedStringPart(yyDollar[1].token.Value)
yyVAL.list = []node.Node{encapsed, yyDollar[2].node}
@@ -8992,7 +8999,7 @@ yydefault:
}
case 495:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6423
//line php5/php5.y:6429
{
name := node.NewIdentifier(strings.TrimLeftFunc(yyDollar[1].token.Value, isDollar))
yyVAL.node = expr.NewVariable(name)
@@ -9008,7 +9015,7 @@ yydefault:
}
case 496:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6437
//line php5/php5.y:6443
{
identifier := node.NewIdentifier(strings.TrimLeftFunc(yyDollar[1].token.Value, isDollar))
variable := expr.NewVariable(identifier)
@@ -9028,7 +9035,7 @@ yydefault:
}
case 497:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6455
//line php5/php5.y:6461
{
identifier := node.NewIdentifier(strings.TrimLeftFunc(yyDollar[1].token.Value, isDollar))
variable := expr.NewVariable(identifier)
@@ -9050,7 +9057,7 @@ yydefault:
}
case 498:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6475
//line php5/php5.y:6481
{
yyVAL.node = expr.NewVariable(yyDollar[2].node)
@@ -9065,7 +9072,7 @@ yydefault:
}
case 499:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6488
//line php5/php5.y:6494
{
name := node.NewIdentifier(yyDollar[2].token.Value)
yyVAL.node = expr.NewVariable(name)
@@ -9083,7 +9090,7 @@ yydefault:
}
case 500:
yyDollar = yyS[yypt-6 : yypt+1]
//line php5/php5.y:6504
//line php5/php5.y:6510
{
identifier := node.NewIdentifier(yyDollar[2].token.Value)
variable := expr.NewVariable(identifier)
@@ -9105,7 +9112,7 @@ yydefault:
}
case 501:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6524
//line php5/php5.y:6530
{
yyVAL.node = yyDollar[2].node
@@ -9113,7 +9120,7 @@ yydefault:
}
case 502:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6533
//line php5/php5.y:6539
{
yyVAL.node = scalar.NewString(yyDollar[1].token.Value)
@@ -9127,7 +9134,7 @@ yydefault:
}
case 503:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6545
//line php5/php5.y:6551
{
// TODO: add option to handle 64 bit integer
if _, err := strconv.Atoi(yyDollar[1].token.Value); err == nil {
@@ -9146,7 +9153,7 @@ yydefault:
}
case 504:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6562
//line php5/php5.y:6568
{
identifier := node.NewIdentifier(strings.TrimLeftFunc(yyDollar[1].token.Value, isDollar))
yyVAL.node = expr.NewVariable(identifier)
@@ -9162,7 +9169,7 @@ yydefault:
}
case 505:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6579
//line php5/php5.y:6585
{
yyVAL.node = expr.NewIsset(yyDollar[3].list)
@@ -9178,7 +9185,7 @@ yydefault:
}
case 506:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6593
//line php5/php5.y:6599
{
yyVAL.node = expr.NewEmpty(yyDollar[3].node)
@@ -9194,7 +9201,7 @@ yydefault:
}
case 507:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6607
//line php5/php5.y:6613
{
yyVAL.node = expr.NewEmpty(yyDollar[3].node)
@@ -9210,7 +9217,7 @@ yydefault:
}
case 508:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6621
//line php5/php5.y:6627
{
yyVAL.node = expr.NewInclude(yyDollar[2].node)
@@ -9224,7 +9231,7 @@ yydefault:
}
case 509:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6633
//line php5/php5.y:6639
{
yyVAL.node = expr.NewIncludeOnce(yyDollar[2].node)
@@ -9238,7 +9245,7 @@ yydefault:
}
case 510:
yyDollar = yyS[yypt-4 : yypt+1]
//line php5/php5.y:6645
//line php5/php5.y:6651
{
yyVAL.node = expr.NewEval(yyDollar[3].node)
@@ -9254,7 +9261,7 @@ yydefault:
}
case 511:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6659
//line php5/php5.y:6665
{
yyVAL.node = expr.NewRequire(yyDollar[2].node)
@@ -9268,7 +9275,7 @@ yydefault:
}
case 512:
yyDollar = yyS[yypt-2 : yypt+1]
//line php5/php5.y:6671
//line php5/php5.y:6677
{
yyVAL.node = expr.NewRequireOnce(yyDollar[2].node)
@@ -9282,7 +9289,7 @@ yydefault:
}
case 513:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6686
//line php5/php5.y:6692
{
yyVAL.list = []node.Node{yyDollar[1].node}
@@ -9290,7 +9297,7 @@ yydefault:
}
case 514:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6692
//line php5/php5.y:6698
{
yyVAL.list = append(yyDollar[1].list, yyDollar[3].node)
@@ -9301,7 +9308,7 @@ yydefault:
}
case 515:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6704
//line php5/php5.y:6710
{
yyVAL.node = yyDollar[1].node
@@ -9309,7 +9316,7 @@ yydefault:
}
case 516:
yyDollar = yyS[yypt-1 : yypt+1]
//line php5/php5.y:6710
//line php5/php5.y:6716
{
yyVAL.node = yyDollar[1].node
@@ -9317,7 +9324,7 @@ yydefault:
}
case 517:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6719
//line php5/php5.y:6725
{
target := node.NewIdentifier(yyDollar[3].token.Value)
yyVAL.node = expr.NewClassConstFetch(yyDollar[1].node, target)
@@ -9333,7 +9340,7 @@ yydefault:
}
case 518:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6733
//line php5/php5.y:6739
{
target := node.NewIdentifier(yyDollar[3].token.Value)
yyVAL.node = expr.NewClassConstFetch(yyDollar[1].node, target)
@@ -9349,7 +9356,7 @@ yydefault:
}
case 519:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6750
//line php5/php5.y:6756
{
target := node.NewIdentifier(yyDollar[3].token.Value)
yyVAL.node = expr.NewClassConstFetch(yyDollar[1].node, target)
@@ -9365,7 +9372,7 @@ yydefault:
}
case 520:
yyDollar = yyS[yypt-3 : yypt+1]
//line php5/php5.y:6767
//line php5/php5.y:6773
{
target := node.NewIdentifier(yyDollar[3].token.Value)
yyVAL.node = expr.NewClassConstFetch(yyDollar[1].node, target)

8
php5/php5.y
View File

@@ -6185,10 +6185,16 @@ simple_indirect_reference:
assignment_list:
assignment_list ',' assignment_list_element
{
if len($1) == 0 {
$1 = []node.Node{nil}
}
$$ = append($1, $3)
// save comments
yylex.(*Parser).comments.AddFromToken(lastNode($1), $2, comment.CommaToken)
if lastNode($1) != nil {
yylex.(*Parser).comments.AddFromToken(lastNode($1), $2, comment.CommaToken)
}
yylex.(*Parser).returnTokenToPool(yyDollar, &yyVAL)
}