-
Notifications
You must be signed in to change notification settings - Fork 44
Open
Description
When running rql_test sometimes the split function gets stuck and gets to a point of out of memory.
After a lot of debugging i found out that the way the equalExp function implemented is wrong.
The check
if s1[i][0] == '(' && s2[j][0] == '(' {
found = equalExp(s1[i][1:len(s1[i])-1], s2[j][1:len(s2[j])-1])
}is wrong because if we get ( at the start we are removing the end of the string, not the next ) found which is the problem
there is a corruption of data (a = b AND c = d) OR (a = c AND (c = d OR d = f)) will be a = b AND c = d) OR (a = c AND (c = d OR d = f)
The problem starts here and get to the split function which is not validating the given data and not breaking out of the while loop after it failes at the start with finding )
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels