Beefy Boxes and Bandwidth Generously Provided by pair Networks
Syntactic Confectionery Delight
 
PerlMonks  

Re^4: Preventing malicious T-SQL injection attacks

by Win (Novice)
on Mar 05, 2007 at 15:34 UTC ( #603231=note: print w/ replies, xml ) Need Help??


in reply to Re^3: Preventing malicious T-SQL injection attacks
in thread Preventing malicious T-SQL injection attacks

Can I also question the need to specify the number of variables that each SPROC can take. Because a SPROC won't allow an execution if an incorrect number of variables is specified.


Comment on Re^4: Preventing malicious T-SQL injection attacks
Re^5: Preventing malicious T-SQL injection attacks
by davorg (Chancellor) on Mar 05, 2007 at 16:05 UTC

    You can question whatever you want. It's your code, after all.

    But you need to know the number of parameters in order to create an SQL string with the correct number of placeholders. So DBI checks the number of parameters for you for free.

    You get an extra layer of defensive programming for no cost. I can't see any reason why you wouldn't want to make use of it.

      I think that it would be good when SELECT is used in a similar circumstance. But when that particular feature is used with EXEC I believe it is redundant code and therefore is best not used.

        But what is redundant? What would you remove? Like I said, this is a completely free feature. There is no code in there which specifically checks for the right number of parameters, it's just something that execute gives you for free.

        There is no redundancy. There is nothing to remove. If you find something to remove then I'd love to see it.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://603231]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others cooling their heels in the Monastery: (8)
As of 2014-09-22 20:59 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    How do you remember the number of days in each month?











    Results (201 votes), past polls