[Вопрос] sscanf string buffer overflow

sscanf string buffer overflow

PHP код:

CMD:makeadmin(playerid, params[]) { if(PI[playerid][pAdmin] < 5) return 0; if(!ADuty{playerid}) return SendClientMessage(playerid, COLOR_GREY, NO_DUTY_TEXT); new id,admin,string[200]; if(sscanf(params,"ud",id,admin)) return SendClientMessage(playerid,COLOR_RED,"Используйте: /makeadmin [id] [level (0 - 6)]"); if(!IsPlayerConnected(id)) return 0; if(GetString(PN(id), !"Mihail_Zubenko") || GetString(PN(id), !"Brian_DeSanto") || GetString(PN(id), !"Van_Kard")) return 0; if(PI[playerid][pAdmin] <= PI[id][pAdmin]) { SendClientMessage(playerid,COLOR_RED,"Отказано в доступе!"); format(string,sizeof(string),"Игрок %s[%d] пытается снять админа %s",PN(playerid),playerid,PN(id)); SendAdminsMessage(COLOR_RED,string); return 0; } if(!GetString(PN(playerid),"Mihail_Zubenko") ) { if(!IsASuccesIP(PlayerRegIP[playerid], PlayerIp[playerid]) && PI[playerid][pAdmin]<7) { SendClientMessage(playerid,COLOR_LIGHTRED,"Наблюдается попытка взлома!"); format(string,sizeof(string),"%s [%d] подозревается во зломе, проверьте его и авторизируйте /acceptadmin [7 lvl]",PN(playerid),playerid); SendAdminsMessage(COLOR_RED,string); return 0; } } if(admin < 0 || admin > 6) return SendClientMessage(playerid,COLOR_VALIK,"Используйте: /makeadmin [id] [level (0 - 6)]"); if(PI[playerid][pAdmin]==5 && admin > 4) return SendAdminsMessage(COLOR_RED,"net!"); PI[id][pAdmin] = admin; format(string,sizeof(string),"Вы выдали игроку %s %d-й уровень администратирования",Name(id),admin); SendClientMessage(playerid,COLOR_LIGHTBLUE,string); format(string,sizeof(string)," %s назначил игрока %s админом %d-го уровня",Name(playerid),Name(id),admin); SendAdminsMessage(COLOR_RED,string); return 1; }

Много ненужного, но как-то так. А лучшее вообще использовать extract.

А вообще, описывать проблему нужно, уважаемый. А еще, перед тем, как создать тему, прочитать правила ее оформления.

PHP код:

CMD:makeadmin(playerid, params[]) { if( ( GetString(PN(playerid), !"Mihail_Zubenko") || GetString(PN(playerid), !"Brian_DeSanto") || GetString(PN(playerid), !"Van_Kard") ) && !ADuty{playerid} ) return true; new id,admin,string[200]; if(sscanf(params,"ud",id,admin)) return SendClientMessage(playerid,COLOR_RED,"Используйте: /makeadmin [id] [level (0 - 6)]"); if(PI[playerid][pAdmin] <= PI[id][pAdmin]) { SendClientMessage(playerid,COLOR_RED,"Отказано в доступе!"); format(string,sizeof(string),"Игрок %s[%d] пытается снять админа %s",PN(playerid),playerid,PN(id)); SendAdminsMessage(COLOR_RED,string); return true; } if(!IsASuccesIP(PlayerRegIP[playerid], PlayerIp[playerid]) && PI[playerid][pAdmin]<7) { SendClientMessage(playerid,COLOR_LIGHTRED,"Наблюдается попытка взлома!"); format(string,sizeof(string),"%s [%d] подозревается во зломе, проверьте его и авторизируйте /acceptadmin [7 lvl]",PN(playerid),playerid); SendAdminsMessage(COLOR_RED,string); return true; } if(admin < 0 || admin > 6) return SendClientMessage(playerid,COLOR_VALIK,"Используйте: /makeadmin [id] [level (0 - 6)]"); if(PI[playerid][pAdmin]==5 && admin > 4) return SendAdminsMessage(COLOR_RED,"net!"); PI[id][pAdmin] = admin; format(string,sizeof(string),"Вы выдали игроку %s %d-й уровень администратирования",Name(id),admin); SendClientMessage(playerid,COLOR_LIGHTBLUE,string); format(string,sizeof(string)," %s назначил игрока %s админом %d-го уровня",Name(playerid),Name(id),admin); SendAdminsMessage(COLOR_RED,string); return 1; }

Цитата:

Сообщение от Fallen A.

Много ненужного, но как-то так. А лучшее вообще использовать extract.

А вообще, описывать проблему нужно, уважаемый. А еще, перед тем, как создать тему, прочитать правила ее оформления.

PHP код:

CMD:makeadmin(playerid, params[]) { if( ( GetString(PN(playerid), !"Mihail_Zubenko") || GetString(PN(playerid), !"Brian_DeSanto") || GetString(PN(playerid), !"Van_Kard") ) && !ADuty{playerid} ) return true; new id,admin,string[200]; if(sscanf(params,"ud",id,admin)) return SendClientMessage(playerid,COLOR_RED,"Используйте: /makeadmin [id] [level (0 - 6)]"); if(PI[playerid][pAdmin] <= PI[id][pAdmin]) { SendClientMessage(playerid,COLOR_RED,"Отказано в доступе!"); format(string,sizeof(string),"Игрок %s[%d] пытается снять админа %s",PN(playerid),playerid,PN(id)); SendAdminsMessage(COLOR_RED,string); return true; } if(!IsASuccesIP(PlayerRegIP[playerid], PlayerIp[playerid]) && PI[playerid][pAdmin]<7) { SendClientMessage(playerid,COLOR_LIGHTRED,"Наблюдается попытка взлома!"); format(string,sizeof(string),"%s [%d] подозревается во зломе, проверьте его и авторизируйте /acceptadmin [7 lvl]",PN(playerid),playerid); SendAdminsMessage(COLOR_RED,string); return true; } if(admin < 0 || admin > 6) return SendClientMessage(playerid,COLOR_VALIK,"Используйте: /makeadmin [id] [level (0 - 6)]"); if(PI[playerid][pAdmin]==5 && admin > 4) return SendAdminsMessage(COLOR_RED,"net!"); PI[id][pAdmin] = admin; format(string,sizeof(string),"Вы выдали игроку %s %d-й уровень администратирования",Name(id),admin); SendClientMessage(playerid,COLOR_LIGHTBLUE,string); format(string,sizeof(string)," %s назначил игрока %s админом %d-го уровня",Name(playerid),Name(id),admin); SendAdminsMessage(COLOR_RED,string); return 1; }

Как использовать extract?

Цитата:

Сообщение от verteich

Как использовать extract?

В гугле забанили?

Не благодари: ссылка на другой ресурс.
Кортез, не обижайся.

Цитата:

Сообщение от Fallen A.

В гугле забанили?

Не благодари: ссылка на другой ресурс.
Кортез, не обижайся.

А в чем отличия extract от обычного sscanf?

Цитата:

Сообщение от verteich

А в чем отличия extract от обычного sscanf?

Как минимум, так удобнее.

Да и код, как минимум, должен быть читабелен.

А по производительности не скажу, ибо PAWN не открывал уже несколько лет подряд.

extract - это просто макрос. По факту это тот же sscanf.

#define extract%0->%1; EXTRN%1;unformat(_:EXTRZ:EXTRV:EXTRX:%0,##,%1,,);

И насчёт читаемости - дело вкуса. Как по мне, обычный sscanf гораздо лаконичнее, ибо и с нативками идентичен (SetTimer/SetTimerEx/CallLocalFunction и т.п. имеют тот же синтаксис), и более выделяется за счёт подсветки строки.

Fallen A., ты в своём варианте, как минимум, забыл проверку на онлайн добавить, из-за чего выход за пределы массива обеспечен. Не говоря уже о том, что у автора проверка на ник идёт для игрока, которому выдаётся админка, а не для админа, который выдаёт.

Автор, версия sscanf последняя? Если нет, то обнови. И вот:

CMD:makeadmin(playerid, params[]) 
{ 
    if(PI[playerid][pAdmin] < 5) 
        return 0; 
    if(!ADuty{playerid}) 
        return SendClientMessage(playerid, COLOR_GREY, NO_DUTY_TEXT); 
 
    new targetid,
        level; 
    if(sscanf(params,"ud", targetid, level)) 
        return SendClientMessage(playerid,COLOR_RED,"Используйте: /makeadmin [targetid] [level (0 - 6)]");
 
    if(targetid == INVALID_PLAYER_ID) 
        return 0; 
 
    if(level < 0 || level > 6) 
        return SendClientMessage(playerid,COLOR_VALIK,"Используйте: /makeadmin [targetid] [level (0 - 6)]"); 
 
    if(PI[playerid][pAdmin]==5 && level > 4) 
        return SendAdminsMessage(COLOR_RED,"net!"); 
 
    if(GetString(PN(targetid), !"Mihail_Zubenko") || GetString(PN(targetid), !"Brian_DeSanto") || GetString(PN(targetid), !"Van_Kard")) 
        return 0; 
 
    new string[145];
    if(PI[playerid][pAdmin] <= PI[targetid][pAdmin]) 
    { 
        SendClientMessage(playerid,COLOR_RED,"Отказано в доступе!"); 
        format(string,sizeof(string),"Игрок %s[%d] пытается снять админа %s", PN(playerid), playerid, PN(targetid)); 
        SendAdminsMessage(COLOR_RED,string); 
        return 0; 
    } 
    if(!GetString(PN(playerid),!"Mihail_Zubenko") ) 
    { 
        if(!IsASuccesIP(PlayerRegIP[playerid], PlayerIp[playerid]) && PI[playerid][pAdmin]<7) 
        { 
            SendClientMessage(playerid,COLOR_LIGHTRED,"Наблюдается попытка взлома!"); 
            format(string,sizeof(string),"%s [%d] подозревается во зломе, проверьте его и авторизируйте /acceptadmin [7 lvl]",PN(playerid),playerid); 
            SendAdminsMessage(COLOR_RED,string); 
            return 0; 
        } 
    } 
    PI[targetid][pAdmin] = level; 
    format(string,sizeof(string),"Вы выдали игроку %s %d-й уровень администратирования",Name(targetid),level); 
    SendClientMessage(playerid,COLOR_LIGHTBLUE,string); 
    format(string,sizeof(string)," %s назначил игрока %s админом %d-го уровня",Name(playerid),Name(targetid),level); 
    SendAdminsMessage(COLOR_RED,string); 
    return 1; 
}

UPD: перенёс последующий оффтоп сюда

решено