Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

OMEMO not sending "long" messages #56

Closed
COM8 opened this issue Feb 28, 2019 · 2 comments
Closed

OMEMO not sending "long" messages #56

COM8 opened this issue Feb 28, 2019 · 2 comments
Labels
Milestone

Comments

@COM8
Copy link
Member

COM8 commented Feb 28, 2019

Bevor you create a new issue:

- [ X ] I searched for similar issues and did not find one
- [ X ] I'm using the latest version available in the [Windows Store](https://www.microsoft.com/store/apps/9NW16X9JB5WV)

I'm submitting a...:

  • Bug report (I searched for similar issues and did not find one)

Current behavior:

If you try to send longer OMEMO encrypted messages they stay in the sending state and never actually get send.

Logs:

Error during message parsing.System.Xml.XmlException: '�', hexadecimal value 0x15, is an invalid character. Line 1, position 1.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.ParseText(Int32& startPos, Int32& endPos, Int32& outOrChars)
   at System.Xml.XmlTextReaderImpl.ParseText()
   at System.Xml.XmlTextReaderImpl.ParseDocumentContent()
   at System.Xml.XmlTextReaderImpl.Read()
   at XMPP_API.Classes.Network.XML.MessageParser2.parseToXmlNodes(String& msg)
   at XMPP_API.Classes.Network.XML.MessageParser2.parseMessageInternal(String& msg)
   at XMPP_API.Classes.Network.XML.MessageParser2.parseMessages(String& msg)
   at XMPP_API.Classes.Network.XMPPConnection2.<parseMessageAsync>d__62.MoveNext()

Expected behavior:

Send long OMEMO encrypted messages.

Minimal reproduction of the problem with instructions:

Send for example:

Lorem ipsum dolor sit amet consectetur adipiscing elit curabitur, a maecenas morbi sodales convallis urna iaculis dui vel, risus accumsan quam odio cubilia integer molestie. Pharetra donec blandit euismod purus aliquam libero dignissim et semper, tempus phasellus volutpat arcu est odio fusce rutrum eu, nulla convallis ad ac laoreet pretium sociosqu tortor. Sagittis bibendum nibh eget tempus integer fermentum interdum molestie taciti sem elementum nisi, gravida et cum hac fusce pharetra potenti a blandit varius maecenas, netus laoreet nam lectus vel praesent vitae urna aenean risus natoque.

Environment:

App Version(s): 
v.0.12.0.0

Windows 10 Version Number: <!-- https://en.wikipedia.org/wiki/Windows_10_version_history -->
- [ ] 1809
- [ ] 1803
- [ ] 1709
- [ ] 1703
- [ ] 1607
- [ ] 1511
- [ ] 1507
- [ ] Insider Build (build number: )
- [ ] Misc:

Device form factor:
- [ X ] Desktop
- [ X ] Mobile
- [ ] Xbox
- [ ] Surface Hub

Where did you got the APP from?
- [ X ] [Windows Store](https://www.microsoft.com/store/apps/9NW16X9JB5WV)
- [ ] Self-build, using a provided release source
- [ ] Self-build, repo cloned at [dd.mm.yyy] <!-- When did you clone the repo! -->
- [ ] Misc, got it from... <!-- Please tell us your source! -->

@COM8 COM8 added the 🐛 Bug label Feb 28, 2019
@COM8 COM8 added this to the Beta ~2019 milestone Feb 28, 2019
@COM8 COM8 changed the title OMEMO not sending "long" messges OMEMO not sending "long" messages Mar 12, 2019
@COM8
Copy link
Member Author

COM8 commented Mar 14, 2019

This issue is caused by:

int AesGcmWrapper::encrypt(WriteOnlyArray<uint8>^ output, WriteOnlyArray<uint8>^ tag, const Array<uint8>^ input, const Array<uint8>^ key, const Array<uint8>^ iv) {

It does not return for inputs greater than 255 bytes.

@COM8
Copy link
Member Author

COM8 commented Mar 14, 2019

openluopworld/aes_gcm#1

@COM8 COM8 closed this as completed in 8f34971 Mar 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant