AKalgutkar148549 (Community Member) asked a question.
How can we fix the flaw in the below line of jsp code
<input type="hidden" id="username" name="username" value="<%= (String) session.getAttribute("username")%>"/>
Ask the Community
Get answers, share a use case, discuss your favorite features, or get input from the community.
By clicking “Accept All Cookies”, you agree to the storing of cookies on your device to enhance site navigation, analyze site usage, and assist in our marketing efforts.
.png){kind=logo}
These cookies are necessary for the website to function and cannot be switched off in our systems. They are usually only set in response to actions made by you which amount to a request for services, such as setting your privacy preferences, logging in or filling in forms. You can set your browser to block or alert you about these cookies, but some parts of the site will not then work. These cookies do not store any personally identifiable information.
These cookies allow us to count visits and traffic sources so we can measure and improve the performance of our site. They help us to know which pages are the most and least popular and see how visitors move around the site. All information these cookies collect is aggregated and therefore anonymous. If you do not allow these cookies we will not know when you have visited our site, and will not be able to monitor its performance.
These cookies may be set through our site by our advertising partners. They may be used by those companies to build a profile of your interests and show you relevant adverts on other sites. They do not store directly personal information, but are based on uniquely identifying your browser and internet device. If you do not allow these cookies, you will experience less targeted advertising.
Hi @AKalgutkar148549 (Community Member),
Veracode Static Analysis reports CWE 201 (Insertion of Sensitive Information Into Sent Data) when it detects that potentially sensitive data is sent out of the application. The concerns being that the sent-out information might be sensitive in itself (such as credentials or private messages) or an attacker might be able to leverage the information for further attacks.
In your scenario, our Static Engine can detect data from the session being sent out (i.e. being displayed in a JSP), in this case, the username. To resolve this finding, we recommend reviewing if the transfer of the data is intended and does not violate your application security policy. If it is unintended or violates your app sec policy, please discontinue sending that particular data out. Otherwise, please propose a mitigation and discuss it with your security team. You can find more information about how to propose mitigations here: https://help.veracode.com/r/improve_mitigation .
Thank you,
Florian Walter
Hi @AKalgutkar148549 (Community Member)
CWE 201 is flagged usually when the scanner believes that one of the potentially sensitive data is being transferred out of the application.
The question we will ask is: Whether this data is really sensitive or not. The scanner might be wrongly identifying it but is basically giving us all an opportunity to review if we may be passing any sensitive data. The scanner usually refers to configuration or properties as "Sensitive", so it is best we review each of such properties used in the flagged code.
If there is no sensitive data, please feel free to propose mitigation (refer: https://help.veracode.com/r/improve_mitigation) by explaining which config or environmental variables are used and why you think they are not sensitive at all. If they are sensitive, ensure that the transfer of that sensitive data is intended and that it does not violate the application security policy. This flaw is categorized as low severity because it only impacts confidentiality, not integrity or availability.
If you still have questions I would recommend you schedule a consultation call to discuss.
You can check out this knowledge article (https://community.veracode.com/s/article/How-to-schedule-a-consultation-call) on how to schedule a consultation call with us.
Regards,
Kashif Syed