Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parse_skype_message_blob issue #11

Open
Banaanhangwagen opened this issue May 17, 2021 · 1 comment
Open

parse_skype_message_blob issue #11

Banaanhangwagen opened this issue May 17, 2021 · 1 comment

Comments

@Banaanhangwagen
Copy link

Let me begin by thanking you for your time and effort in this project!

When trying -m of your script on a Skype for Desktop (v8.46), it gave the following error:

Skype1

Apparently, the db I want to parse has some values that parse_skype_message_blob does not accept and it stops.

After commenting them out, it works as expected.

image

This is what I changed:

	// first field is a Varint, maybe the record ID
	parseVarInt(&p, pend);
	// expect 0xff
	// assert(*p == 0xff);
	p++;

	// expect 0x12
	//assert(*p == 0x12 || *p == 0x13 || *p == 0x14);
	p++;

	// expect 0xff
	//assert(*p == 0xff);
	p++;

	// expect 0x0d
	//assert(*p == 0x0d);
	p++;

This is the (modified) output of the 4 first entries which causes the error:
Skype4

Unfortunately, I am not able to share the db with you.

Can this be patched in your code?
Thank you!

@rtravis
Copy link
Owner

rtravis commented May 18, 2021

Thanks for your feedback. I will test with more databases and hopefully I can come up with a fix. If not then I'll just comment out the lines that you suggested. By the way, do all the asserts fail when the first one does?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants