Skip to content

Commit

Permalink
Correct access checks in mc_issue_get_id_from_summary
Browse files Browse the repository at this point in the history
Fixes #13736: mc_issue_get_id_from_summary incorrectly checks for
permissions
  • Loading branch information
rombert committed Mar 3, 2012
1 parent dea7e31 commit 4dd69a5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion api/soap/mc_issue_api.php
Expand Up @@ -491,7 +491,8 @@ function mc_issue_get_id_from_summary( $p_username, $p_password, $p_summary ) {
$t_issue_id = (int) $row['id'];
$t_project_id = bug_get_field( $t_issue_id, 'project_id' );

if( mci_has_readonly_access( $t_user_id, $t_project_id ) ) {
if( mci_has_readonly_access( $t_user_id, $t_project_id ) &&
access_has_bug_level( VIEWER, $t_issue_id, $t_user_id ) ) {
return $t_issue_id;
}
}
Expand Down

0 comments on commit 4dd69a5

Please sign in to comment.