Changeset 1439

Show
Ignore:
Timestamp:
02/03/08 15:36:24 (7 months ago)
Author:
Neal
Message:

Added validation to prevent users from subscribing to non-existent topics. Discovered by lie2815.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • branches/punbb-1.3-dev/upload/misc.php

    r1375 r1439  
    421421                message($lang_common['Bad request']); 
    422422 
     423        // Make sure the user can view the topic 
     424        $query = array( 
     425                'SELECT'        => '1', 
     426                'FROM'          => 'topics AS t', 
     427                'JOINS'         => array( 
     428                        array( 
     429                                'LEFT JOIN'             => 'forum_perms AS fp', 
     430                                'ON'                    => '(fp.forum_id=t.forum_id AND fp.group_id='.$pun_user['g_id'].')' 
     431                        ) 
     432                ), 
     433                'WHERE'         => '(fp.read_forum IS NULL OR fp.read_forum=1) AND t.id='.$topic_id.' AND t.moved_to IS NULL' 
     434        ); 
     435        ($hook = get_hook('mi_qr_subscribe_topic_exists')) ? eval($hook) : null; 
     436        $result = $db->query_build($query) or error(__FILE__, __LINE__); 
     437        if (!$db->num_rows($result)) 
     438                message($lang_common['Bad request']); 
     439 
    423440        $query = array( 
    424441                'SELECT'        => '1',