Skip to content
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

CFQUERYPARAM_REQ false positive: String in SQL Server XML function #297

Closed
TheRealAgentK opened this issue Jun 20, 2017 · 7 comments
Closed
Assignees

Comments

@TheRealAgentK
Copy link
Collaborator

...
UPDATE
					Something
				SET
					taskXML.modify('insert #additionalMember#
									   as last into (/task)[1]')
				WHERE
					tID = <cfqueryparam value="#arguments.something.tID#" cfsqltype="cf_sql_integer">
					AND taskUUID = <cfqueryparam value="#arguments.something.tUUID#" cfsqltype="cf_sql_varchar">
...

Reports on the taskXML.modify (taskXML is a column from a preceeding IF EXISTS ( SELECT... ))

@ryaneberly
Copy link
Contributor

@TheRealAgentK , can you expand on this one a bit please. I get a single CFQUERYPARAM_REQ because of #additionalMember# as I would expect.

@TheRealAgentK
Copy link
Collaborator Author

Will try to create and then push an isolated test case in dev.

@TheRealAgentK
Copy link
Collaborator Author

Ah, scratch that - I see what you mean now. Yes, that is a valid one because .modify() actually acts like an update on that XML blurb. It's another one that's not fixable with CFQUERYPARAM though and would benefit from the more fine-grained ignoring in SQL.

@ryaneberly
Copy link
Contributor

Thank you.

@zspitzer
Copy link

zspitzer commented Jul 29, 2017

this is also triggered when the case of the <cfquery> and <cfqueryparam> tags don't match, which is only stylistically messy but still valid

@TheRealAgentK
Copy link
Collaborator Author

TheRealAgentK commented Jul 29, 2017

Hey @zspitzer - can you provide a (couple of) test samples for us and ideally create a new github issue for this?

@zspitzer
Copy link

done #380

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants