Beefy Boxes and Bandwidth Generously Provided by pair Networks
Pathologically Eclectic Rubbish Lister

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

Replies are listed 'Best First'.
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?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://603231]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others studying the Monastery: (4)
As of 2018-06-21 18:35 GMT
Find Nodes?
    Voting Booth?
    Should cpanminus be part of the standard Perl release?

    Results (118 votes). Check out past polls.