News for hackers. Stuff that matters.

A comment on Echo – Assembly program that prints the first positional argument to stdout. Parent comment

Re:Echo – Assembly program that prints the first positional ... by gens on Monday January 02, 2017 @05:27AM (#13300331)
I don't understand why it is limited to 255 chars. The kernel copies the string(s) into the programs memory so it would be a kernel bug if the program got a non null-terminated or too long string.

More importantly this program has a bug in that it doesn't check if there is an argument passed to it at all.

Good effort but can improve a lot. I would praise the documentation but it is rather imprecise. All in all i wouldn't put it on the front page of HN yet.

[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by kelseyhightower on Monday January 02, 2017 @08:42AM (#13301095)
This is great feedback, which I plan to use to improve the echo program. I'm just learning (on my own), and I figured I would just post my progress and I would get some feedback; it worked!

echo is far from finished, and it's safe to say "I don't know what the hell I'm doing", but hey, I gotta start somewhere.

[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by unixbhaskar on Monday January 02, 2017 @11:20AM (#13301985)
That's the spirit Kelsey! keep it up.
[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by pavlov on Monday January 02, 2017 @06:03AM (#13300438)
Maybe the 255-char limit is a feature? If this "fast echo" is meant to be used in a script that writes entries to a log where you wouldn't want long text anyway, or something like that... So having a known upper bound for the output size can be useful.
[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by cyphar on Monday January 02, 2017 @06:59AM (#13300638)
> I don't understand why it is limited to 255 chars. The kernel copies the string(s) into the programs memory so it would be a kernel bug if the program got a non null-terminated or too long string.

But you can also pass arguments to execve(2) which are not null-terminated.

[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by gens on Monday January 02, 2017 @06:08PM (#13305288)
The kernel copies the strings you pass in the array of pointers. (haven't checked though, but it is better then the alternative of not copying and dealing with the mess)
[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by cyphar on Tuesday January 03, 2017 @06:28PM (#13314282)
The memory mapping is the same before and after execve(2), so I don't think it needs to be copied. I'll take a look though.
[ Parent | on HN ]
Re:Echo – Assembly program that prints the first positional ... by gens on Wednesday January 04, 2017 @09:41AM (#13318577)
Maybe only a few pages remain as programs don't inherit memory from their parents. It could be done for those strings but consider that mappings are in 4k pages (so the rest of the page would have to be cleared to 0).
[ Parent | on HN ]