bartobri / no-more-secrets

A command line tool that recreates the famous data decryption effect seen in the 1992 movie Sneakers.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

'sneakers' command only works once

byamj opened this issue · comments

commented

Executing the sneakers command works the first time, but running it a second time in the same iTerm session throws Abort trap: 6. Quitting and restarting iTerm allows it to be run one more time before throwing the same error.

This based on window size, I wasn't careful when attempting to reproduce.

Executing the sneakers command works with window width 242 but not 243 and throws Abort trap: 6. Number gathered from resizing my iTerm window and looking at the status bar. Height doesn't seem to matter.

Edit the second: It's not the absolute size, it seems to be the ratio between height and width that causes the error.

System:
macOS Sierra version 10.12.6

@byamj , do you experience this issue with a smaller terminal size? I think it is the sheer size of your terminal that is causing my program to write past the buffer limit. It's a legit bug on my end. But first let me know if you can reproduce with smaller terminal sizes.

Thanks.

commented

With a window height of 2, 246 is the widest the window can be without causing an error. Window height 60 also works at 246 width but not 247 so I think that's the magic number.

This is fixed in commit 56df5f3 and pushed to the master branch.

@byamj can you pull the latest code from the master branch, recompile, and confirm that it is fixed for you?

Thanks!

commented

Code is pulled but unfortunately I am working remote today and don't have access to a monitor where I can make my terminal that size. Will test tomorrow, sorry for the delay but wanted to say something!

commented

Confirmed to be fixed at width 247 through 270/fullscreen.