Beefy Boxes and Bandwidth Generously Provided by pair Networks
We don't bite newbies here... much

Re^5: Preventing malicious T-SQL injection attacks

by davorg (Chancellor)
on Mar 05, 2007 at 16:05 UTC ( #603239=note: print w/replies, xml ) Need Help??

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

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.

  • Comment on Re^5: Preventing malicious T-SQL injection attacks

Replies are listed 'Best First'.
Re^6: Preventing malicious T-SQL injection attacks
by Win (Novice) on Mar 05, 2007 at 16:31 UTC
    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.

        I suppose that it would add an extra layer of security given that, otherwise, a hacker may manage to alter the SPROC to take an extra variable and then run that without changes to the Perl program.

Log In?

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

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (7)
As of 2018-06-20 21:34 GMT
Find Nodes?
    Voting Booth?
    Should cpanminus be part of the standard Perl release?

    Results (117 votes). Check out past polls.