The code had an obvious bug when memory was accessed out of bounds. In some of the cases it was accessing zeroes and producing some output, but after my changes it started to work "as designed" and use "space" (not zero) as padding.
I'm not sure what is the best way to fix this. Changing padding symbol from "space" to zero should also make decryption working. I really do not know what is the best solution and what is better "to be right" or "to be backward compatible". Best regards, Petro On Thu, Aug 11, 2022, 10:10 PM Alan Carvalho de Assis <acas...@gmail.com> wrote: > ACK > > Strange, the previous email went only to you! > > On 8/11/22, Alin Jerpelea <jerpe...@gmail.com> wrote: > > @Alan Carvalho de Assis <acas...@gmail.com> please confirm that works > > after revert > > > > On Thu, 11 Aug 2022, 20:22 Petro Karashchenko, > > <petro.karashche...@gmail.com> > > wrote: > > > >> Hello Alan, > >> > >> Seems that the root cause is my change > >> https://github.com/apache/incubator-nuttx-apps/pull/1097 > >> > >> I think that all previously generated passwords need to be re-generated. > >> > >> Best regards, > >> Petro > >> > >> On Thu, Aug 11, 2022, 8:49 PM Alan Carvalho de Assis <acas...@gmail.com > > > >> wrote: > >> > >> > Alin, > >> > > >> > I want to redraw my vote, I found the first regression, so my vote is: > >> > > >> > -1 > >> > > >> > Seems like the console login is not working, I'm using user: admin and > >> > password: Administrator > >> > > >> > $ ./tools/configure.sh sim:nsh > >> > > >> > $ make -j > >> > > >> > $ ./nuttx > >> > login: admin > >> > password: > >> > Invalid username or password > >> > login: admin > >> > password: > >> > Invalid username or password > >> > login: admin > >> > password: > >> > Invalid username or password > >> > Login failed! > >> > > >> > I double checked the /etc/passwd file and it is correct: > >> > > >> > nsh> cat /etc/passwd > >> > admin:8Tv+Hbmr3pLddSjtzL0kwC:0:0:/ > >> > > >> > Please help me to find the offending commit. > >> > > >> > BR, > >> > > >> > Alan > >> > > >> > On 8/8/22, Alin Jerpelea <jerpe...@gmail.com> wrote: > >> > > Hello all, > >> > > Apache NuttX (Incubating) 10.4.0 RC0 has been staged under [1] and > >> > > it's > >> > > time to vote on accepting it for release. If approved we will seek > >> > > final release approval from the IPMC. Voting will be open for 72hr. > >> > > > >> > > A minimum of 3 binding +1 votes and more binding +1 than binding -1 > >> > > are > >> > > required to pass. > >> > > > >> > > The Apache requirements for approving a release can be found here > [3] > >> > > "Before voting +1 [P]PMC members are required to download the signed > >> > > source code package, compile it as provided, and test the resulting > >> > > executable on their own platform, along with also verifying that the > >> > > package meets the requirements of the ASF policy on releases." > >> > > > >> > > A document to walk through some of this process has been published > on > >> > > our project wiki and can be found here [4]. > >> > > > >> > > [ ] +1 accept (indicate what you validated - e.g. performed the > >> > > non-RM > >> > > items in [4]) > >> > > [ ] -1 reject (explanation required) > >> > > > >> > > Thank you all, > >> > > Alin Jerpelea > >> > > > >> > > SCM Information: > >> > > Release tag: nuttx-10.4.0-RC0 > >> > > Hash for the release incubating-nuttx tag: > >> > > dd718e78f70f9350ac648067509672c5051841b9 > >> > > Hash for the release incubating-nuttx-apps tag: > >> > > 8b43f9f9ca30f44c1cccae9a9078d5d45b776d35 > >> > > > >> > > [1] > >> > > https://dist.apache.org/repos/dist/dev/incubator/nuttx/10.4.0-RC0/ > >> > > [2] > >> > > >> > https://raw.githubusercontent.com/apache/incubator-nuttx/nuttx-10.4.0-RC0/ReleaseNotes > >> > > [3] https://www.apache.org/dev/release.html#approving-a-release > >> > > [4] > >> > > >> > https://cwiki.apache.org/confluence/display/NUTTX/Validating+a+staged+Release > >> > > > >> > > >> > > >
