Почему разный результат байт?

Question

[Ярослав Иванов]

Использую online_c++_compiler

Почему разный результат?

Вывод консоли

���:������������������������������������T���H���U��G���s���r���v���V���2��� ���T���A�

и

:THUGsrvV2 TA+CMManhattan�i�\����8�]�����8�]�]H������X��
�]�|

Как видно в первом варианите есть буквы T H U G, которые разделяются �

Код, который это генерирует:

Код

#include <iostream>
#include <cstring>

using namespace std;

int main()
{
  int out_len;
	unsigned char key;
	int i;
	char packet_data[100];
    
  int in_stream[100] = { 202, 240, 218, 42, 202, 202, 194, 202, 202, 202, 203, 203, 202, 158, 130, 159, 141, 185, 184, 188, 156, 248, 234, 158, 139, 225, 137, 135, 202, 135, 171, 164, 162, 171, 190, 190, 171, 164, 202, 74, 163, 114, 207, 150, 203, 2, 59, 5, 202, 121, 242, 97, 151, 202, 202, 202, 202, 98, 62, 5, 202, 202, 59, 5, 202, 15, 242, 97, 151, 221, 213, 100, 151, 130, 62, 5, 202, 54, 53, 53, 53, 146, 62, 5, 202, 192, 215, 100, 151, 202, 202, 202, 202, 200, 202, 202, 202, 68, 182, 207 };
  out_len = 99;
    
  memcpy( packet_data, in_stream, out_len );

	key = in_stream[0];

	for( i = 0; i < out_len; i++ )
	{
		packet_data[i] = (in_stream[i]) ^ key; 
                // Если менять in_stream на packet_data то получаем первый вариант
                // Если оставляем то - второй.
		
		//cout<<i;
		//cout<<":";
		cout<<packet_data[i];
		//cout<<"\n";
	}

    return 0;
}

Как правильно декодировать пакет? Настройки какие нужны?

Answer 1

[rPman]

У тебя разные типы данных и соответственно размеры int это 4 байта (или 8 в зависимости от компилятора и архитектуры) и char - 1 байт
int in_stream[100]
и
char packet_data[100];

memcpy работает с байтами а значит копирует только часть данных

Comments

[Ярослав Иванов]

Как исправить?)

[rPman]

Ярослав Иванов, подумать
зачем у скопированного тобой бездумно кода тип элементов instream - int?

[Ярослав Иванов]

rPman, может надо все int сделать? packet_data и key? Все операции сделать а потом байты привести к строке?

[rPman]

зачем тебе int?

[Ярослав Иванов]

rPman, это байты

[rPman]

Int по 4 байта

[Ярослав Иванов]

rPman, пока не наводит на мысли

[rPman]

еще раз задаю вопрос почему у тебя packet_data это массив char а in_stream это массив int? зачем тебе копировать мороженное в котлеты?

[Ярослав Иванов]

rPman, "может надо все int сделать? packet_data и key?"

[rPman]

попробуй и проверь результат
у меня вопрос, ты разницу между int и char понимаешь?

[Ярослав Иванов]

rPman, Вы можете по факту ответить?

[rPman]

какая задача?
в вопросе дан только код с ошибкой но не сказано что нужно получить, я конечно могу телепатствовать но зачем

[Ярослав Иванов]

rPman, "Как правильно декодировать пакет?"
Есть ощущение, что при конвертации лишние символы. Или это норма пакета. Данных мало так как игре больше 20 лет.

[Ярослав Иванов]

rPman,
:THUGsrvV2 TA+CMManhattan�i�\����8�]�����8�]�]H������X��
�]�|
Вывод похож на правду так как тут корректное название сервера "THUGsrvV2 TA+CM" и название карты "Manhattan" Но вот что остальное - без понятия. И это консоль вывод сократила. Там в начале еще несколько пробелов.

API протокола и доков нет)

[Ярослав Иванов]

Это UDP пакет

[rPman]

в твоем ошибочном виде
T���H���U��G���s���r���v���V���2���тебе ничего не намекает?

символы вопросы - это нули

in_stream[100] = { 202, 240, 218, 42,вот тут что ты делаешь? что это за числа?

[Ярослав Иванов]

rPman, "это байты"

[rPman]

отлично почему ты байты пихаешь в int массив?

[Ярослав Иванов]

rPman, а куда надо?

[Ярослав Иванов]

rPman, поставил uint8_t
Получил
THUGsrvV2 TA+CMManhattan�i�\����8�]�����8�]�]H������X��
�]�|
Уже лучше

[rPman]

ну вот формально верно
правда хватило бы и char но писать как ты вернее так как ты гарантируешь что это будет именно байт

[Ярослав Иванов]

rPman, Ок что тогда есть остальные символы? Разве пакет может быть с мусором? Он же корректно мне конвертит первую часть.

[Ярослав Иванов]

rPman, Вот полностью пример от которого я отталкивался (уже поменял int на uint8_t)

int actual_crc;
	int out_len;
	unsigned char key;
	unsigned short hi_word, lo_word;
	unsigned short packet_crc, sum_crc;
	unsigned char* in;
	int i;
	char packet_data[ 1300 ];
    int in_len = 100;
    uint8_t in_stream[100] = { 202, 240, 218, 42, 202, 202, 194, 202, 202, 202, 203, 203, 202, 158, 130, 159, 141, 185, 184, 188, 156, 248, 234, 158, 139, 225, 137, 135, 202, 135, 171, 164, 162, 171, 190, 190, 171, 164, 202, 74, 163, 114, 207, 150, 203, 2, 59, 5, 202, 121, 242, 97, 151, 202, 202, 202, 202, 98, 62, 5, 202, 202, 59, 5, 202, 15, 242, 97, 151, 221, 213, 100, 151, 130, 62, 5, 202, 54, 53, 53, 53, 146, 62, 5, 202, 192, 215, 100, 151, 202, 202, 202, 202, 200, 202, 202, 202, 68, 182, 207 };

	out_len = in_len - sizeof( unsigned short );
	memcpy( &packet_crc, in_stream, sizeof( unsigned short ));
	memcpy( packet_data, in_stream + sizeof( unsigned short ), out_len );	// skip CRC

	key = (unsigned char) packet_crc;	// use the lsB of the packet CRC to decrypt the packet
	in = (unsigned char*) packet_data;
	for( i = 0; i < out_len; i++ )
	{
		*in = (*in) ^ key;
		in++;
		
		//cout<<i;
		//cout<<"::";
		cout<<packet_data[i];
		//cout<<"\n";
	}

Зачем in++; ? это же ссылка на значение?

[Ярослав Иванов]

rPman, А взял я тип char из аргументов. Этот код выше был упакован в

validate_and_copy_stream( char* in_stream, char* out_stream, int in_len )
я вижу char и сделал массив char'ов. А оно вон как работает.

Значение in_stream получает от recvfrom()

[Ярослав Иванов]

rPman, А может пакет надо читать по-разному?

Где-то как char, где-то как uint8_t а где-то uint16_t ?
Поэтому повезло, что одни строки конвертнулись а другая часть пакета нет?

[rPman]

оп а сразу сказать что тебе именно надо?
чтобы разбирать пакет данных с элементами разного типа у тебя 2 подхода
* первый - заведи свой тип данных в виде структуры, описывающей весь пакет твоих данных (к сожалению это подходит только для тех типов пакетов, структура которых не имеет динамических блоков, например строк переменной длины, или их мало)
struct MyPacket
{
uint16_t myFirstElement;
uint8_t[16] myFixedLenName;
...
}
тогда твой буфер - это буквально переменная этого типа, и работать с ней можно как есть, в си есть не только простые типы но и битовые (переменная являющаяся несколькими битами от куска памяти) и union, когда один и то же блок в структуре может быть на выбор разными элементами, лишь бы размер был одинаковый.
недостаток, компиляторы по своей прихоти, ради оптимизации могут добавлять в структуру выравнивания и даже менять порядок, у разных компиляторов есть соответствующие директивы

#pragma pack(push, 1) // это отключит выравнивание по 32-64 битам

* второй - держи пакет данных в виде массива байт без типа void* (хотя любой другой подойдет, но удобнее байтовый тип), затем по шагам ты берешь нужное количество байт из этого массива и сдвигаешь указатель на соответствующее количество байт, размером с этот элемент, пример:

*(uint16_t*)buf; // читаем данные
buf+=sizeof(uint16_t); // делаем сдвиг

этот подход самый гибкий, но нужно следить за выходом за границы, так как преобразования типов ненадежны, в смысле если ты допустишь ошибку, ты можешь об этом не сразу узнать.

[Ярослав Иванов]

rPman, вынести в ответы