Permalink
Browse files

Additional patch to fix #14016

First patch to fix the issue introduced access to undefined variable.
  • Loading branch information...
1 parent 628e937 commit c9314184f541f0e3e3b91b3533104e50292c3e68 @atrol atrol committed Jun 3, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 api/soap/mc_issue_attachment_api.php
@@ -72,7 +72,7 @@ function mc_issue_attachment_delete( $p_username, $p_password, $p_issue_attachme
$t_bug_id = file_get_field( $p_issue_attachment_id, 'bug_id' );
# Perform access control checks
- $t_attachment_owner = file_get_field( $f_file_id, 'user_id' );
+ $t_attachment_owner = file_get_field( $p_issue_attachment_id, 'user_id' );
$t_current_user_is_attachment_owner = $t_attachment_owner == $t_user_id;
# Factor in allow_delete_own_attachments=ON|OFF
if ( !$t_current_user_is_attachment_owner || ( $t_current_user_is_attachment_owner && !config_get( 'allow_delete_own_attachments' ) ) ) {

0 comments on commit c931418

Please sign in to comment.