Changeset 5647


Ignore:
Timestamp:
Sep 11, 2017 10:31:03 AM (7 years ago)
Author:
riza
Message:

Re #1994(misc): Fixed unguarded 'if' which might lead to double locking. Thanks to Alexander Traud for the info.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • pjproject/trunk/pjlib-util/src/pjlib-util/cli_telnet.c

    r5311 r5647  
    14921492        case ST_CR: 
    14931493            sess->parse_state = ST_NORMAL; 
    1494             if (*cdata == 0 || *cdata == '\n') 
     1494            if (*cdata == 0 || *cdata == '\n') {                 
    14951495                pj_mutex_unlock(sess->smutex); 
    14961496                is_valid = handle_return(sess); 
     
    14981498                    return PJ_FALSE; 
    14991499                pj_mutex_lock(sess->smutex); 
    1500                 break; 
     1500            } 
     1501            break; 
    15011502        case ST_NORMAL: 
    15021503            if (*cdata == IAC) { 
Note: See TracChangeset for help on using the changeset viewer.