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

RFC_864_pattern is not correct with xinetd TCP server #7292

Closed
jeromecoutant opened this issue Jun 21, 2018 · 2 comments
Closed

RFC_864_pattern is not correct with xinetd TCP server #7292

jeromecoutant opened this issue Jun 21, 2018 · 2 comments

Comments

@jeromecoutant
Copy link
Collaborator

Description

I set up a local TCP server in order to execute netsocket tests.

I am using xinetd, and configured services as recommened:

discard stream tcp6 nowait root internal
discard dgram udp6 wait root internal
echo stream tcp6 nowait root internal
echo dgram udp6 wait root internal
chargen stream tcp6 nowait root internal
chargen dgram udp6 wait root internal
daytime stream tcp6 nowait root internal
time stream tcp6 nowait root internal

Then, there are some issues with the chargen service in TCPSOCKET_RECV_100K sub test:

static void generate_RFC_864_pattern(size_t offset, uint8_t *buf, size_t len)

After several checks, it seems there are several misalignments with what inetd service is providing:

  • Pattern starts with !, not space

  • 2 characters are missing : ~ and space

  • sometimes, 2 lines are repeated : (maybe every 94*74 characters)

!"#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\]^_`abcdefgh
"#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\]^_`abcdefghi
!"#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\]^_`abcdefgh
"#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\]^_`abcdefghi

I think current xinetd code is wrong…
https://github.com/xinetd-org/xinetd/blob/e7c1ba41f4f86b436fb82b0d55cd5d387bd4ecc4/xinetd/builtins.c#L377

But mbed-os test has to be updated with the current situation...

Issue request type

[ ] Question
[x] Enhancement
[ ] Bug

@adbridge
Copy link
Contributor

adbridge commented Oct 4, 2018

Internal Jira reference: https://jira.arm.com/browse/ONME-3733

@SeppoTakalo
Copy link
Contributor

Fix has been submitted in #8645 so this can be closed.

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

No branches or pull requests

5 participants