Greetings Denys, > On Thu, Feb 8, 2018 at 1:55 PM, daggs <da...@gmx.com> wrote: > > I've tested the implementation on 1.28.0 with the two following patches: > > https://git.busybox.net/busybox/commit/shell/hush.c?id=b8d076b00bad9da87e6dcaf4f36dfd06e2d2e5e8 > > https://git.busybox.net/busybox/commit/shell/hush.c?id=6aad1ddcc9d8fe2303cc444b6ddea2af1fc1dd30 > > on the test script I've posted in the first mail and the output is as > > follows: > > $ ./busybox hush test.sh > > hush: using fallback suid method > > at line 5 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 6 > > at line 10 > > hello world 10 > > > > that differs from the bash output, the line that is printed most is > > actually in line 7 and not 6. > > I think this implementation suffers from a similar but my implementation > > suffered. > > > Reproduced. > > while ....; do > echo ... > done > > hush parser thinks that the command is "do echo ..." :) and therefore > it starts on line-1. > > Fixed in git, please try now. >
indeed that worked but there is another scenario it goes haywire. I'll try to provide a test case script that reproduces it soon. Thanks for all the hard work, Dagg. _______________________________________________ busybox mailing list busybox@busybox.net http://lists.busybox.net/mailman/listinfo/busybox