KirankumarBharsadiya (Community Member) asked a question.

I got medium vulnerable flaw of "Improperly Controlled Modification of Dynamically-Determined Object Attributes" under title of "Insufficient Input Validation".

I got the issue after static scan for my C# ASP.NET Framework application. But the location given in report are not proper.

Locations with the line number suggest only the constrctor of the controllers, not any specific thing. So how to tackle this vulnerability in

my application. Is there any code structure that I can match with my code and find some lead to handle it.


  • Hi @KirankumarBharsadiya (Community Member)​ ,

     

    Veracode Static Analysis should not report constructors for CWE 915, though sometimes missing debugging symbols or the use of async constructs (for which the compiler generates a lot of constructs) may make it difficult for us to give you the correct line number from the binary at is may appear to report the first line, corresponding to the constructor.

     

    Please look at the detailed description for this flaw in the Veracode Platform and verify that the 'Attack Vector' as well as the 'Data Path' both point to the constructor.

     

    If this points to the constructor, please create a case with Veracode Support through the link in the footer of the Veracode Platform ('Contact Support') or through the Veracode Community in the menu on the top right after clicking on your avatar ('Contact Support') and provide the flaw id as well as a link to the Triage Flaws view in the Veracode Platform where the flaw can be found.

     

    Otherwise this should provide you with details for this flaw.

     

    Veracode Static Analysis only automatically detects use of [Bind(Include="Property1,Property2")] (see also: https://learn.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.mvc.bindattribute.include?view=aspnetcore-7.0 ) as remediation (automatically closes the flaw upon rescan).

    You may also chose to use a "ViewModel" or "Data Transfer Object (DTO)" pattern where all properties are designed to be bound to but we will not automatically recognize this and you must then manually document this in a mitigation proposal. You can learn more about this here: https://docs.veracode.com/r/improve_mitigation . This must then be reviewed by someone in your organization with the 'Mitigation Approver' right.

     

    Please let me know if you have any remaining questions or concerns.

     

    Thank you,

    Boy Baukema

    Expand Post

Topics (1)

No articles found
Loading

Ask the Community

Get answers, share a use case, discuss your favorite features, or get input from the community.