Ru-MaNGOS - PACKED данные

Ru-MaNGOS (http://mangos.ytdb.ru/index.php)

- Опкоды (http://mangos.ytdb.ru/forumdisplay.php?f=36)

- - PACKED данные (http://mangos.ytdb.ru/showthread.php?t=1824)

PACKED данные

как известно некоторые опкоды приходят не по одному, а пачками, как например SMSG_COMPRESSED_MOVES, COMPRESSED_UPDATE_OBJECT...
Где можно глянуть как распаковать такие данные ? или подкиньте кусок кода.

Дак тамже простейший zlib.
Или ты не про это?

Про то.
Просто я никогда не использовал zlib библиотеку и не знаю что там с чем едят

пусть я буду некропостером, но блин, из-за "простейший zlib" я пол дня мозг выносил с проблемой декодирования. Хорошо потом стал проверять, отсекая первые байты.
что то вроде:
1. uint32 - размер данных после декодирования
2. 2 байта относятся к zlib , никакой полезной нагрузки не несут.
3. остальное - сжатые данные, без заголовков

зы в подтверждение у TOM_RUS-а увидел распаковку :D

ну да, так и есть, сперва идет DWORD - размер данных после распаковки, а потом кусок данных который нужно отдать для обработки функции inflate
Я тогда сразу написал класс для распаковки:

Код:

bool CZlib::decompress(BYTE* in_buffer, int in_size, int out_size)

{

        m_out_buffer = new BYTE[out_size];

        if (!m_out_buffer)

        {

                return false;

        }

        m_zstream.next_in = in_buffer;

        m_zstream.avail_in = in_size;

        m_zstream.next_out = m_out_buffer;

        m_zstream.avail_out = out_size;

        if (inflateInit(&m_zstream) == Z_OK)

        {

                if (inflate(&m_zstream, Z_FINISH) == Z_STREAM_END)

                {

                        return true;

                }

                inflateEnd(&m_zstream);

        }

        return false;

}

И использовался он так:

Код:

void CDataHandler::on_SMSG_COMPRESSED_UPDATE_OBJECT(BYTE* _packet, DWORD _len)

{

        int unpacked_len = 0;

        memcpy(&unpacked_len, _packet, 4);

        CZlib zlib;

        if (zlib.decompress(_packet + 4, _len - 4, unpacked_len))

        {

                parse_update(zlib.get_data());

        }

}

спросил бы :)
все правильно сделал. только inflateEnd надо еще на Z_OK проверять.

Я сейчас точно не помню, но судя по своему же коду вызов inflateEnd не нужен, ибо возврат из функции идет сразу после проверки (inflate(&m_zstream, Z_FINISH) == Z_STREAM_END)
насколько я помню документацию на злиб, то если вызов inflate завершился кодом Z_STREAM_END, то данные полностью распакованы.
В данном случае поскольку нам известна заранее длина распакованных данных, то можно сразу выделять буфер нужной длины.
Что у меня в коде и делается, так что строчку с inflateEnd надо удалить нафиг -)

представь, что будет, если тебе придется обрабатывать битый пакет или пакет с ошибками Lordronn =)
проверять надо всё, включая переполнения буфера и inflateEnd.

и вообще строго говоря, кто сказал что надо сразу inflate(&m_zstream, Z_FINISH) вызывать? пакеты с ачивками в волке очень большие могут быть. размер m_zstream.avail_out делать больше одной страницы - не по феншую как бэ... но можно. в спецификации там WORD был вроде (тоже не помню), но я пробовал и longword в полмега и распаковывается все за один раз без проблем в принципе. но в таких случаях могут возникать ошибки stack overflow когда прога разрастется и хрен ты их найдешь сходу =)

ну в идеале конечно да

а мы разве не к нему стремимся?

Цитата:

Сообщение от RomanRom2 (Сообщение 22120)

и вообще строго говоря, кто сказал что надо сразу inflate(&m_zstream, Z_FINISH) вызывать? пакеты с ачивками в волке очень большие могут быть. размер m_zstream.avail_out делать больше одной страницы - не по феншую как бэ... но можно. в спецификации там WORD был вроде (тоже не помню), но я пробовал и longword в полмега и распаковывается все за один раз без проблем в принципе. но в таких случаях могут возникать ошибки stack overflow когда прога разрастется и хрен ты их найдешь сходу =)

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