From 7b438e28893867a526e070ef172aa63a0440566f Mon Sep 17 00:00:00 2001
From: johnnyq <johnny@pittpc.com>
Date: Mon, 2 Mar 2026 17:38:20 -0500
Subject: [PATCH] Transfers: Add missing CSRF checks

---
 agent/modals/transfer/transfer_add.php  | 2 ++
 agent/modals/transfer/transfer_edit.php | 1 +
 agent/post/transfer.php                 | 6 ++++++
 agent/transfers.php                     | 2 +-
 4 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/agent/modals/transfer/transfer_add.php b/agent/modals/transfer/transfer_add.php
index 63aec004..1a885219 100644
--- a/agent/modals/transfer/transfer_add.php
+++ b/agent/modals/transfer/transfer_add.php
@@ -12,6 +12,8 @@ ob_start();
     </button>
 </div>
 <form action="post.php" method="post" autocomplete="off">
+    <input type="hidden" name="csrf_token" value="<?= $_SESSION['csrf_token'] ?>">
+
     <div class="modal-body">
 
         <div class="form-row">
diff --git a/agent/modals/transfer/transfer_edit.php b/agent/modals/transfer/transfer_edit.php
index 4305496e..7956a5c0 100644
--- a/agent/modals/transfer/transfer_edit.php
+++ b/agent/modals/transfer/transfer_edit.php
@@ -33,6 +33,7 @@ ob_start();
     </button>
 </div>
 <form action="post.php" method="post" autocomplete="off">
+    <input type="hidden" name="csrf_token" value="<?= $_SESSION['csrf_token'] ?>">
     <input type="hidden" name="transfer_id" value="<?php echo $transfer_id; ?>">
     <input type="hidden" name="expense_id" value="<?php echo $expense_id; ?>">
     <input type="hidden" name="revenue_id" value="<?php echo $revenue_id; ?>">
diff --git a/agent/post/transfer.php b/agent/post/transfer.php
index 2915f543..170f7e00 100644
--- a/agent/post/transfer.php
+++ b/agent/post/transfer.php
@@ -8,6 +8,8 @@ defined('FROM_POST_HANDLER') || die("Direct file access is not allowed");
 
 if (isset($_POST['add_transfer'])) {
 
+    validateCSRFToken($_POST['csrf_token']);
+
     enforceUserPermission('module_financial', 2);
 
     require_once 'transfer_model.php';
@@ -43,6 +45,8 @@ if (isset($_POST['add_transfer'])) {
 
 if (isset($_POST['edit_transfer'])) {
 
+    validateCSRFToken($_POST['csrf_token']);
+
     enforceUserPermission('module_financial', 2);
 
     require_once 'transfer_model.php';
@@ -67,6 +71,8 @@ if (isset($_POST['edit_transfer'])) {
 
 if (isset($_GET['delete_transfer'])) {
 
+    validateCSRFToken($_GET['csrf_token']);
+
     enforceUserPermission('module_financial', 3);
 
     $transfer_id = intval($_GET['delete_transfer']);
diff --git a/agent/transfers.php b/agent/transfers.php
index bc4a08e5..6d91576b 100644
--- a/agent/transfers.php
+++ b/agent/transfers.php
@@ -226,7 +226,7 @@ $num_rows = mysqli_fetch_row(mysqli_query($mysqli, "SELECT FOUND_ROWS()"));
                                             <i class="fas fa-fw fa-edit mr-2"></i>Edit
                                         </a>
                                         <div class="dropdown-divider"></div>
-                                        <a class="dropdown-item text-danger text-bold confirm-link" href="post.php?delete_transfer=<?= $transfer_id ?>">
+                                        <a class="dropdown-item text-danger text-bold confirm-link" href="post.php?delete_transfer=<?= $transfer_id ?>&csrf_token=<?= $_SESSION['csrf_token'] ?>">
                                             <i class="fas fa-fw fa-trash mr-2"></i>Delete
                                         </a>
                                     </div>