PSangani109845 (Community Member) asked a question.
For correction, I added regular expression validation to allow only certain file extensions while uploading files, but it is still not resolving.
but giving an error at delete file method.
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,
It is nice that you are implementing the Regular Expression based validation to allow only specific set of Extensions for file uploading and I am assuming the same file handle is used for Delete() where CWE 73 is reported.
We require additional mitigation to limit the risk of CWE 73 (External Control of Filename or Path) .
I would suggest you to do a canonical file path checking in addition to above.
In canonical checking, you will compare the Starting of the file path against set of known target paths.
I highly recommend you to read this article :
https://community.veracode.com/s/article/how-do-i-fix-cwe-73-external-control-of-file-name-or-path-in-java
Also please see the article on Allow-list approach and see how it can be useful to mitigate CWE 73.
https://community.veracode.com/s/article/allowlist-helps-fix-cwe
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
Hi @PSangani109845 (Community Member)
It is nice that you are implementing the Regular Expression based validation to allow only specific set of Extensions for file uploading and I am assuming the same file handle is used for Delete() where CWE 73 is reported.
We require additional mitigation to limit the risk of CWE 73 (External Control of Filename or Path) .
I would suggest you to do a canonical file path checking in addition to above.
In canonical checking, you will compare the Starting of the file path against set of known target paths.
I highly recommend you to read this article :
https://community.veracode.com/s/article/how-do-i-fix-cwe-73-external-control-of-file-name-or-path-in-java
Also please see the article on Allow-list approach and see how it can be useful to mitigate CWE 73.
https://community.veracode.com/s/article/allowlist-helps-fix-cwe
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