Files
codeql/csharp/ql/test/query-tests/Security Features/CWE-099/ResourceInjection.cs
2026-07-07 14:09:48 +01:00

30 lines
1.1 KiB
C#

using System.Data.SqlClient;
using System.Web;
public class ResourceInjectionHandler : IHttpHandler
{
public void ProcessRequest(HttpContext ctx)
{
string userName = ctx.Request.QueryString["userName"]; // $ Source=r1 Source=r2
string connectionString = "server=(local);user id=" + userName + ";password= pass;";
// BAD: Direct use of user input in a connection string for the constructor
SqlConnection sqlConnection = new SqlConnection(connectionString); // $ Alert=r1 Alert=r2
// BAD: Direct use of user input assigned to a connection string property
sqlConnection.ConnectionString = connectionString; // $ Alert=r1 Alert=r2
// GOOD: Use SqlConnectionStringBuilder
SqlConnectionStringBuilder builder = new SqlConnectionStringBuilder();
builder["Data Source"] = "(local)";
builder["integrated Security"] = true;
builder["user id"] = userName;
SqlConnection sqlConnectionGood = new SqlConnection(builder.ConnectionString);
}
public bool IsReusable
{
get
{
return true;
}
}
}