-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stack overflow in dev branch #293
Comments
It must have to do something with a particular file, because if I run it on dev against the component sub-directory, it executes fine. But it's a codebase with thousands of files - without being able to figure out which file it's scanning when it runs into an issue, I have no chance. |
If the debugging code in front of the stacktrace loop is supposed to be where the issue occurs, it'd be in Application.cfc, in an onSessionStart() function.
|
Is there a cflinclude there? It is looping on the cfinclude lookup. |
@TheRealAgentK See if the latest from the dev branch resolves your issue |
Yes, it does, yay! I do get a couple of: "Terminated a recursive call to include file /Users/kai/Documents/Code/... something.cfm" now. I assume that's how you fixed it? |
Was it a legitimate loop? Is there really a recursive include in your code?
…On Sun, Jun 18, 2017 at 8:27 PM, Kai Koenig ***@***.***> wrote:
Closed #293 <#293>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#293 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFtZu7yHdQ7XgGvNFoMyVlVBDmQyhBHSks5sFcBigaJpZM4N2UCm>
.
|
I've looked at some of the instances - all is very ugly/old and messed up code. One is in x.inc.cfm:
So, it's including itself and I struggle to even get the context of what is being achieved there - something with reordering blog comments. The other scenario involves CFLocation and then a CFInclude and I that doesn't look like a recursion situation to me at first glance, but I need to dig into what that code is actually doing. |
Running
java -jar CFLint-1.1.0-all.jar -folder /Users/kai/Documents/Code/somefolder -xml -xmlfile something.xml -xmlstyle findbugs -includeRule CFQUERYPARAM_REQ,QUERYPARAM_REQ,MISSING_VAR -v -e -stats
throws this in the dev branch and then exits:
Tried a few changesets back in dev and it happened on all of them, I went back up to including 71066e5 so far.
The text was updated successfully, but these errors were encountered: