Skip to content

heimdall: Odin protocol version 4 support#459

Open
rtammekivi wants to merge 2 commits intoBenjamin-Dobell:masterfrom
rtammekivi:master
Open

heimdall: Odin protocol version 4 support#459
rtammekivi wants to merge 2 commits intoBenjamin-Dobell:masterfrom
rtammekivi:master

Conversation

@rtammekivi
Copy link
Copy Markdown

I investigated with Odin3 and wireshark to see the differences. Odin versions 3.12.xx sent version 3, and the newer ones send version 4. This is needed for S8/S8+ american variants it seems.

Heimdall thus far did not send version number at all.

@rtammekivi
Copy link
Copy Markdown
Author

This is successfully tested with S6 and upwards, different models.

{
SessionSetupPacket::Pack();

/* Odin protocol verison. */
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a cosmetic thingy, but there is spelling mistake "verison" in the comment

Copy link
Copy Markdown
Author

@rtammekivi rtammekivi Mar 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, I will update. Do you otherwise think this is ok or is there anything else to change?
Besides testing of course

@rtammekivi
Copy link
Copy Markdown
Author

I updated the pull request, I'm not sure if I did it properly though.

@Benjamin-Dobell
Copy link
Copy Markdown
Owner

@Blefish Thanks for this.

Before I merge I'll have to find some time to dust off all my old Samsung devices (pre S6) and see how this affects them. Also have an S8 on hand so will test there as well.

@rtammekivi
Copy link
Copy Markdown
Author

Sounds good to me, I would like to know if if actually fixes the problem for someone else as well, because this commit is from december and I used older Heimdall version to base this on originally.

@swedishjeff
Copy link
Copy Markdown

swedishjeff commented Apr 17, 2018

I'll bite. I'm seeing a successful (?) output from heimdall:

Initialising connection...
Detecting device...
Claiming interface...
Setting up interface...

Initialising protocol...
Protocol initialisation successful.

Beginning session...
Releasing device interface...

But I get the error "Odin protocol version error" on the S7 I'm working with.

If I can get the updated version for Linux (I don't know how to build from source) I'll give it a try.

@nvtkaszpir
Copy link
Copy Markdown

@swedishjeff about linux build, see Linux/README at the very end.

@fftmp
Copy link
Copy Markdown

fftmp commented Mar 28, 2021

This fix worked for me also. Why it doesn't accepted?

Percentage fix: fftmp-forked@04e8eb6

@ixonstater
Copy link
Copy Markdown

This fix worked for me as well on a Samsung S7.

@KTibow
Copy link
Copy Markdown

KTibow commented Aug 18, 2022

This fix doesn't work for me:

Heimdall v1.4.2

Copyright (c) 2010-2017 Benjamin Dobell, Glass Echidna
http://www.glassechidna.com.au/

This software is provided free of charge. Copying and redistribution is
encouraged.

If you appreciate this software and you would like to support future
development please consider donating:
http://www.glassechidna.com.au/donate/

Initialising connection...
Detecting device...
Claiming interface...
Setting up interface...

Initialising protocol...
ERROR: Protocol initialisation failed!

Releasing device interface...

With --no-reboot --resume, the pit it outputs is empty.

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

Successfully merging this pull request may close these issues.

10 participants