Другое С/С++ Вопрос - Ответ

[rraggerr]

Есть функция для поиска последовательности строчных символов:

int findstring(void* where_, char* what, int whereSize)
{
    void* temp = malloc(strlen(what) + 1);
    int iterator = 0;
    char* yo = new char[strlen(what) + 1];
    yo = (char*)temp;
    char* tempptr = static_cast<char*>(where_);
    while (iterator <= whereSize)
    {
        memcpy(temp, tempptr + iterator, strlen(what));
        yo[strlen(what) + 1] = 0;
        if (strcmp((char*)temp, what) == 0)
        {
            free(temp);
            //delete yo;
            return iterator;
        }
            iterator++;
    }
    return -1;
}

Она копирует из области поиска по n байт длиною равной длине искомого, добавляет на конец 0 и сравнивает. И так, пока не кончится буфер.
Однако, я её раз 10 наверное прогнал по циклу вызвов с одинаковыми параметрами, и примерно 33-39 из 100 вызовов она возвращает ошибку(-1). Лично я тут не вижу ошибки в логике, но результат говорит об обратном.
Помогите понять, где косяк, плиз.

это проверка на то что "слово" есть в строчке "какое то слово" ?

 

[ШPEK]

#include "stdafx.h"
#include <Windows.h>
#include <process.h>
#include <stdio.h>


#define REGISTER_CLIENT_COMMAND 0x65AC0
#define ADD_CHAT_MESSAGE 0x64520
#define pChat 0x21A0E4
#define pInput 0x21A0E8
#define memory_info 0x8A5A80
DWORD dwSamp;

void AddChatMessage(const char* msg)
{
    ((void(__thiscall *)(DWORD str, const char* text))(dwSamp + ADD_CHAT_MESSAGE))(*(DWORD*)(dwSamp + pChat), msg);
}

bool isInit()
{
    return ((bool(__thiscall *)())(dwSamp + 0x9BF70))();
}

void AddClientCommand(const char* cmd, void* func)
{
    ((void (__thiscall *)(DWORD str, const char* cmd, void* func))(dwSamp + REGISTER_CLIENT_COMMAND))(*(DWORD*)(dwSamp + pInput), cmd, func);
}

void core()
{
    char buf[40];
    sprintf_s(buf, "%u", *(DWORD*)(dwSamp + memory_info));
    AddChatMessage(buf);
}

void attach()
{
    dwSamp = (DWORD)GetModuleHandle(L"samp.dll");
    while (!dwSamp) {
        dwSamp = (DWORD)GetModuleHandle(L"samp.dll");
        Sleep(30);
    }
    dwSamp = (DWORD)GetModuleHandle(L"samp.dll");
    AddClientCommand("getmem", core);
}


BOOL APIENTRY DllMain(HMODULE hModule,
    DWORD  ul_reason_for_call,
    LPVOID lpReserved
)
{
    if (ul_reason_for_call == DLL_PROCESS_ATTACH) _beginthread((_beginthread_proc_type)attach, NULL, NULL);
    return TRUE;
}

Крашит игру. в MoonLoader'e проверял вызов AddChatMessage, всё работало. AddChatMessage и AddClientMessage -
это методы класса.

 

[astap_]

Подскажите, если кто-нибудь знает, как работает это? Какие адреса чистит, где их можно узнать итд.

обычный игнор входящего RPC_SCRREMOVEBUILDINGFORPLAYER

 

[ishi]

это проверка на то что "слово" есть в строчке "какое то слово" ?

Не совсем. Это для поиска в буфере, содержимое которого может не соответствовать строчному типу: на конце может не быть нуля, а сами символы могут не соответствовать кодировке искомого. Короче говоря: поиск в бинарном файле.

чем strstr не устраивает?

Не хочет искать дальше первого \0

 

[-raymond-]

как сделать альтернативу keyhook_keycombo_pressed из собейта в сф?

 

[ishi]

Обновляю запрос.

Спойлер: old post

Есть функция для поиска последовательности строчных символов:

int findstring(void* where_, char* what, int whereSize)
{
    void* temp = malloc(strlen(what) + 1);
    int iterator = 0;
    char* yo = new char[strlen(what) + 1];
    yo = (char*)temp;
    char* tempptr = static_cast<char*>(where_);
    while (iterator <= whereSize)
    {
        memcpy(temp, tempptr + iterator, strlen(what));
        yo[strlen(what) + 1] = 0;
        if (strcmp((char*)temp, what) == 0)
        {
            free(temp);
            //delete yo;
            return iterator;
        }
            iterator++;
    }
    return -1;
}

Она копирует из области поиска по n байт длиною равной длине искомого, добавляет на конец 0 и сравнивает. И так, пока не кончится буфер.
Однако, я её раз 10 наверное прогнал по циклу вызвов с одинаковыми параметрами, и примерно 33-39 из 100 вызовов она возвращает ошибку(-1). Лично я тут не вижу ошибки в логике, но результат говорит об обратном.
Помогите понять, где косяк, плиз.

В yo была утечка, исправив слепил это:

int findstring(void* where_, char* what, int whereSize)
{
    void* temp = malloc(strlen(what) + 1);
    int iterator = 0;
    char* yo = static_cast<char*>(temp) + strlen(what) + 1;
    char* tempptr = static_cast<char*>(where_);
    while (iterator <= whereSize)
    {
        memcpy(temp, tempptr + iterator, strlen(what));
        *yo = 0;
        if (strcmp((char*)temp, what) == 0)
        {
            free(temp);
            return iterator;
        }
            iterator++;
    }
    return -1;
}

Теперь из десяти запусков по 100 вызовов, 8-9 запусков идут без ошибок, а конкретно: ошибка вылетает первые <10 вызовов в первом, реже - во втором цикле вызовов.
Проблема определённо связана с памятью, но эта рандомность ошибок ставит меня в ступор. Где ещё? Что конфликтует? Неясная поляна.
Запрос помощи всё ещё актуален.

 

[ALF]

Обновляю запрос.
В yo была утечка, исправив слепил это:

int findstring(void* where_, char* what, int whereSize)
{
    void* temp = malloc(strlen(what) + 1);
    int iterator = 0;
    char* yo = static_cast<char*>(temp) + strlen(what) + 1;
    char* tempptr = static_cast<char*>(where_);
    while (iterator <= whereSize)
    {
        memcpy(temp, tempptr + iterator, strlen(what));
        *yo = 0;
        if (strcmp((char*)temp, what) == 0)
        {
            free(temp);
            return iterator;
        }
            iterator++;
    }
    return -1;
}

Теперь из десяти запусков по 100 вызовов, 8-9 запусков идут без ошибок, а конкретно: ошибка вылетает первые <10 вызовов в первом, реже - во втором цикле вызовов.
Проблема определённо связана с памятью, но эта рандомность ошибок ставит меня в ступор. Где ещё? Что конфликтует? Неясная поляна.
Запрос помощи всё ещё актуален.

в отладчике посмотри

 

[_Vine_]

Обновляю запрос.
В yo была утечка, исправив слепил это:

int findstring(void* where_, char* what, int whereSize)
{
    void* temp = malloc(strlen(what) + 1);
    int iterator = 0;
    char* yo = static_cast<char*>(temp) + strlen(what) + 1;
    char* tempptr = static_cast<char*>(where_);
    while (iterator <= whereSize)
    {
        memcpy(temp, tempptr + iterator, strlen(what));
        *yo = 0;
        if (strcmp((char*)temp, what) == 0)
        {
            free(temp);
            return iterator;
        }
            iterator++;
    }
    return -1;
}

Теперь из десяти запусков по 100 вызовов, 8-9 запусков идут без ошибок, а конкретно: ошибка вылетает первые <10 вызовов в первом, реже - во втором цикле вызовов.
Проблема определённо связана с памятью, но эта рандомность ошибок ставит меня в ступор. Где ещё? Что конфликтует? Неясная поляна.
Запрос помощи всё ещё актуален.

У тебя 3 ошибки в коде.

char* yo = static_cast<char*>(temp) + strlen(what) + 1;

+ 1 не нужен, так ты будешь получать указатель на конец строки + 1, а это не только может приводить к неправильному результату функции, но и изменять данные в чужой выделенной памяти и приводить к сбою программы

while (iterator <= whereSize)

Тут должен быть оператор <, а не <=, ибо так ты будешь сверять символы включая терминальный ноль, а это приведет к неправильному результату
И перед return -1 ты забыл освобождать выделенную память temp
То есть твоя функция должна выглядеть так

int findstring(void* where_, char* what, int whereSize) {
    void* temp = malloc(strlen(what) + 1);
    int iterator = 0;
    char* yo = static_cast<char*>(temp) + strlen(what);
    char* tempptr = static_cast<char*>(where_);
    while (iterator < whereSize) {
        memcpy(temp, tempptr + iterator, strlen(what));
        *yo = 0;
        if (strcmp((char*)temp, what) == 0) {
            free(temp);
            return iterator;
        }
        iterator++;
    }
    free(temp);
    return -1;
}

И вообще, можно было бы реализовать функцию и без промежуточного буфера temp, не знаю зачем ты это сделал.
Или вовсе использовать strstr, раз уже используешь strcmp.

int findstring(const char *str1, const char *str2) {
    int str1_len = strlen(str1);
    int str2_len = strlen(str2);
    for (int i = 0; i < str1_len; i++) {
        bool compare = true;
        for (int k = 0; i < str2_len; k++) {
            if (str1[i + k] != str2[k]) {
                compare = false;
                break;
            }
        }
        if (compare)
            return i;
    }
    return -1;
}

 

[enziweee]

в общем проблема, пытаюсь отрендерить линию от головы до пуза, в итоге линия строится нормально только на первого попавшегося в радиусе стрима актера, на остальных выходит кривота, как исправлять?

Спойлер: code

for (int i = 0; i < MAX_PLAYERS; i++)
        {
            CPlayerPool * pPlayerPool = pNetGame->GetPlayerPool();
            CRemotePlayer * pRemote;

            if (pPlayerPool->GetPlayer(i))
            {
                pRemote = pPlayerPool->GetPlayer(i);
                if (pRemote->m_pPed)
                {
                    SAMP::CPed * pRemotePed = pRemote->m_pPed; // Handle REMOTE Player
                    SAMP::CPed * pLocalPed = SAMP::pGame->m_pPlayerPed; // Handle LOCAL Player

                    if (!pRemotePed->IsDead() || !pLocalPed->IsDead())
                    {
                        SAMP::CVector  pRemoteFirstBoneVector;
                        pRemotePed->GetBonePosition(BONE_HEAD, &pRemoteFirstBoneVector);

                        SAMP::CVector pRemoteSecondBoneVector;
                        pRemotePed->GetBonePosition(BONE_SPINE1, &pRemoteSecondBoneVector);

                        D3DXVECTOR3 pRemoteFirstBoneDX;
                        pRemoteFirstBoneDX.x = pRemoteFirstBoneVector.x;
                        pRemoteFirstBoneDX.y = pRemoteFirstBoneVector.y;
                        pRemoteFirstBoneDX.z = pRemoteFirstBoneVector.z;

                        D3DXVECTOR3 pRemoteSecondBoneDX;
                        pRemoteSecondBoneDX.x = pRemoteSecondBoneVector.x;
                        pRemoteSecondBoneDX.y = pRemoteSecondBoneVector.y;
                        pRemoteSecondBoneDX.z = pRemoteSecondBoneVector.z;

                        D3DXVECTOR3 firstCalculated;
                        D3DXVECTOR3 secondCalculated;

                        CalcScreenCoors(&pRemoteFirstBoneDX, &firstCalculated);
                        CalcScreenCoors(&pRemoteSecondBoneDX, &secondCalculated);

                        draw.Line(firstCalculated.x, firstCalculated.y, secondCalculated.x, secondCalculated.y, D3DCOLOR_ARGB(255, 255, 255, 255));
                        //draw.Text(screenPosEx.x, screenPosEx.y, nickname, fontColor, false, TextAlignment::kCenter);
                    }
                }
            }
        }

Спойлер: Скрины

 

[rraggerr]

в общем проблема, пытаюсь отрендерить линию от головы до пуза, в итоге линия строится нормально только на первого попавшегося в радиусе стрима актера, на остальных выходит кривота, как исправлять?

Спойлер: code

for (int i = 0; i < MAX_PLAYERS; i++)
        {
            CPlayerPool * pPlayerPool = pNetGame->GetPlayerPool();
            CRemotePlayer * pRemote;

            if (pPlayerPool->GetPlayer(i))
            {
                pRemote = pPlayerPool->GetPlayer(i);
                if (pRemote->m_pPed)
                {
                    SAMP::CPed * pRemotePed = pRemote->m_pPed; // Handle REMOTE Player
                    SAMP::CPed * pLocalPed = SAMP::pGame->m_pPlayerPed; // Handle LOCAL Player

                    if (!pRemotePed->IsDead() || !pLocalPed->IsDead())
                    {
                        SAMP::CVector  pRemoteFirstBoneVector;
                        pRemotePed->GetBonePosition(BONE_HEAD, &pRemoteFirstBoneVector);

                        SAMP::CVector pRemoteSecondBoneVector;
                        pRemotePed->GetBonePosition(BONE_SPINE1, &pRemoteSecondBoneVector);

                        D3DXVECTOR3 pRemoteFirstBoneDX;
                        pRemoteFirstBoneDX.x = pRemoteFirstBoneVector.x;
                        pRemoteFirstBoneDX.y = pRemoteFirstBoneVector.y;
                        pRemoteFirstBoneDX.z = pRemoteFirstBoneVector.z;

                        D3DXVECTOR3 pRemoteSecondBoneDX;
                        pRemoteSecondBoneDX.x = pRemoteSecondBoneVector.x;
                        pRemoteSecondBoneDX.y = pRemoteSecondBoneVector.y;
                        pRemoteSecondBoneDX.z = pRemoteSecondBoneVector.z;

                        D3DXVECTOR3 firstCalculated;
                        D3DXVECTOR3 secondCalculated;

                        CalcScreenCoors(&pRemoteFirstBoneDX, &firstCalculated);
                        CalcScreenCoors(&pRemoteSecondBoneDX, &secondCalculated);

                        draw.Line(firstCalculated.x, firstCalculated.y, secondCalculated.x, secondCalculated.y, D3DCOLOR_ARGB(255, 255, 255, 255));
                        //draw.Text(screenPosEx.x, screenPosEx.y, nickname, fontColor, false, TextAlignment::kCenter);
                    }
                }
            }
        }

Спойлер: Скрины

попробуй добавить проверку на
if (CalcScreenCoors)
и на
GetBonePosition
и еще у тебя ошибки, кароче на

if (pLocalPed->IsDead())
return;
for (int i = 0; i <= 1000; i++)
        {
            CPlayerPool * pPlayerPool = pNetGame->GetPlayerPool();
            CRemotePlayer * pRemote;
if (pPlayerPool && pPlayerPool->GetPlayer(i))
            {
                pRemote = pPlayerPool->GetPlayer(i);
                if (pRemote->m_pPed)
                {
                    SAMP::CPed * pRemotePed = pRemote->m_pPed; // Handle REMOTE Player
                    SAMP::CPed * pLocalPed = SAMP::pGame->m_pPlayerPed; // Handle LOCAL Player

                    if (pRemotePed && pLocalPed && !pRemotePed->IsDead() )
                    {
                        SAMP::CVector  pRemoteFirstBoneVector;
                        if (!pRemotePed->GetBonePosition(BONE_HEAD, &pRemoteFirstBoneVector))
continue;

                        SAMP::CVector pRemoteSecondBoneVector;
                       if (!pRemotePed->GetBonePosition(BONE_SPINE1, &pRemoteSecondBoneVector))
                           continue;

                        D3DXVECTOR3 pRemoteFirstBoneDX;
                        pRemoteFirstBoneDX.x = pRemoteFirstBoneVector.x;
                        pRemoteFirstBoneDX.y = pRemoteFirstBoneVector.y;
                        pRemoteFirstBoneDX.z = pRemoteFirstBoneVector.z;

                        D3DXVECTOR3 pRemoteSecondBoneDX;
                        pRemoteSecondBoneDX.x = pRemoteSecondBoneVector.x;
                        pRemoteSecondBoneDX.y = pRemoteSecondBoneVector.y;
                        pRemoteSecondBoneDX.z = pRemoteSecondBoneVector.z;

                        D3DXVECTOR3 firstCalculated;
                        D3DXVECTOR3 secondCalculated;

                       if (!CalcScreenCoors(&pRemoteFirstBoneDX, &firstCalculated))
                           continue;
                       if (CalcScreenCoors(&pRemoteSecondBoneDX, &secondCalculated))
                           continue;

                        draw.Line(firstCalculated.x, firstCalculated.y, secondCalculated.x, secondCalculated.y, D3DCOLOR_ARGB(255, 255, 255, 255));
                        //draw.Text(screenPosEx.x, screenPosEx.y, nickname, fontColor, false, TextAlignment::kCenter);
                    }
                }
            }
        }

 

[enziweee]

попробуй добавить проверку на
if (CalcScreenCoors)
и на
GetBonePosition

каким образом................

 

[rraggerr]

каким образом................

скопируй код и вставь, я не ебу что у тебя там за сдк, явно не сампфункс

 

[enziweee]

скопируй код и вставь, я не ебу что у тебя там за сдк, явно не сампфункс

лучара сдк

 

[Шурик]

// Возникает ошибка

string data;
CHAR szBuffer[1024];
strcpy(szBuffer, "{\"hash\":\"475d03f623908a13b6d8171221119a77\"}");
data += szBuffer;

// Не возникает ошибка

string data;
CHAR szBuffer[1024];
strcpy(szBuffer, "TEST");
data += szBuffer;

Хелп

 

[Stiopko]

// Возникает ошибка

string data;
CHAR szBuffer[1024];
strcpy(szBuffer, "{\"hash\":\"475d03f623908a13b6d8171221119a77\"}");
data += szBuffer;

// Не возникает ошибка

string data;
CHAR szBuffer[1024];
strcpy(szBuffer, "TEST");
data += szBuffer;

Хелп

Попробуй заменить CHAR на char

все норм робит

Спойлер