fixes UB in `void removeQuotes(char **_str)` implemented in src/strings.cpp

3dem / relion

Image-processing software for cryo-electron microscopy

https://relion.readthedocs.io/en/latest/

GNU General Public License v2.0

456 stars 203 forks source link

fixes UB in `void removeQuotes(char **_str)` implemented in src/strings.cpp #1201

Closed Arpan3323 closed 1 week ago

Arpan3323 commented 1 week ago

This pull request addresses #1200

What has been changed:

added null checks for _str and fixed overflow runtime error caused by back quote checking. I wanted to throw REPORT_ERROR on null check for _str but wasn't sure what could be a suitable message. Please let me know if you have any questions or suggestions.