radareorg / radare2

UNIX-like reverse engineering framework and command-line toolset
https://www.radare.org/
GNU Lesser General Public License v3.0
20.71k stars 3.01k forks source link

Implement lock command #6814

Closed radare closed 5 years ago

radare commented 7 years ago

idea from @condret

lock password

the command will not be stored in the history and will clear the screen and ask for password. it will not unlock the console until the password matches.

Maijin commented 7 years ago

That sounds really useless . . . CC @XVilka

Maijin commented 7 years ago

https://github.com/radare/radare2/issues/5132 is useful, lock is not.

XVilka commented 7 years ago

Ehm. What for?

XVilka commented 7 years ago

There are plenty of external tools for locking screen or particular terminal, for all kind of operating systems. I don't see a reason why it should be implemented in radare2.

radare commented 7 years ago

The same goes for having ls, cat.. its useful when u are using it not in your comfy desktop

On 19 Feb 2017, at 12:21, Anton Kochkov notifications@github.com wrote:

There are plenty of external tools for locking screen or particular terminal, for all kind of operating systems. I don't see a reason why it should be implemented in radare2.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

radare commented 7 years ago

Also opens the door to have tutorials about bypassing password protection as a crackme

On 19 Feb 2017, at 12:21, Anton Kochkov notifications@github.com wrote:

There are plenty of external tools for locking screen or particular terminal, for all kind of operating systems. I don't see a reason why it should be implemented in radare2.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

Maijin commented 7 years ago

Well that's just a if/else and there is already an issue open for that iirc.

Maijin commented 7 years ago

Can we lock the number of issues instead...

radare commented 7 years ago

you can fix something to improve the situation :P

On 19 Feb 2017, at 15:58, Maijin notifications@github.com wrote:

Can we lock the number of issues instead...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/radare/radare2/issues/6814#issuecomment-280924331, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3-lje0aJXB6RVyb0GflyAl9ymbkVCDks5reFh8gaJpZM4MFVVn.

Maijin commented 7 years ago

I'm trying :P

radare commented 7 years ago

useful when using r2 as init replacement

Maijin commented 7 years ago

what

XVilka commented 7 years ago

Can be this closed?

radare commented 7 years ago

Nope. Seems handy for me. It must ignore ^z. And also ive been thinking in adding file and process lock commands. Something like "wait untils this file exists", or wait until a process with this name exists.. some kind of wait+test shell

Maijin commented 5 years ago

I've seen this has been done.