Merge pull request #3467 from chenuduss/v3/master

Minor edit with remove useless checks
This commit is contained in:
Ervin Hegedus
2025-11-20 14:08:15 +01:00
committed by GitHub
2 changed files with 17 additions and 17 deletions

View File

@@ -580,7 +580,7 @@ int Multipart::process_part_data(std::string *error, size_t offset) {
m_mpp->m_tmp_file->Open();
/* do we have an opened file? */
if (!m_mpp->m_tmp_file || m_mpp->m_tmp_file->getFd() < 0) {
if (m_mpp->m_tmp_file->getFd() < 0) {
ms_dbg_a(m_transaction, 1,
"Multipart: Failed to create file: " \
+ m_mpp->m_tmp_file->getFilename());

View File

@@ -1312,7 +1312,7 @@ int Transaction::processLogging() {
this->m_rules->evaluate(modsecurity::LoggingPhase, this);
/* If relevant, save this transaction information at the audit_logs */
if (m_rules != NULL && m_rules->m_auditLog != NULL) {
if (m_rules->m_auditLog != NULL) {
int parts = this->m_rules->m_auditLog->getParts();
ms_dbg(8, "Checking if this request is suitable to be " \
"saved as an audit log.");