PureBasic Forums - English

http://www.purebasic.com
https://www.purebasic.fr/english/

Windows Cryptor - completed w/source

https://www.purebasic.fr/english/viewtopic.php?t=43141

Page 3 of 4

Re: Windows Cryptor - wip

Posted: Sat Aug 14, 2010 11:54 am
by Vera
Hello netmaetro,

I love your about-box :) and the task list has become much better to read.

Sorry, to disturb your peace, but I found something unwanted. You can't decrypt files if the full original path is not provided anymore.
You may move the crypted files somewhere else and decrypt them. If the old path still exists they will be moved to that old place. If the old path is gone or changed the decryption fails without a hint why, nor does your programm rebuild the path by itself.

regards ~ Vera

ps: I just see that you've edited your last posting so I didn't have the 'very' last edition, but I guess the discribed circumstance has not coincidentally changed.

Re: Windows Cryptor - wip

Posted: Sat Aug 14, 2010 12:37 pm
by netmaestro
Vera, excellent observation! You know I never thought of that :oops: but the whole reason a lot of people encrypt is so their files are safe in travel. You can't expect the same path structure to exist at the destination and you can't rebuild it if the drive doesn't exist. I don't think it should be too hard to fix, perhaps store just the filepart of the full path in the encryption and then use GetCurrentDirectory()+filepart to create the restored file. Thanks for helping me with that, that's a serious bug that I wasn't going to notice anytime soon. :mrgreen:

Re: Windows Cryptor - wip

Posted: Sat Aug 14, 2010 12:44 pm
by infratec
Hi netmaestro,

one other thing:

Why you don't compress the files additionally?
Because they grow a bit with your encryption.
(a small text file from 1546 to 4371 bytes)
I think since no one can do something with the encryted files,
you can also compress them after or before encryption.

Best regards,

Bernd

Ok, it is only a small fixed number of bytes of growing :D
But still it would be nice if you compress the files.

Re: Windows Cryptor - wip

Posted: Sat Aug 14, 2010 12:53 pm
by infratec
Hi netmaestro,

it's again me.
Why you show always 'Hide' and 'Show' in the menue?
Only one of them is possible. So why you dont replace the text
and check the text, or check your internal flag.
This makes it easier for the user.
Less is more :mrgreen:

Bernd

Re: Windows Cryptor - wip

Posted: Sat Aug 14, 2010 1:02 pm
by infratec
An annoying question:

is it so unsafe to use, for example, 7-Zip with a password?
It uses also a
Strong AES-256 encryption
when you use a password.

And so I don't need a memory stick for the key, I can use my brain memory for the password.

So what was your intention?
To make it easier for the user?

Best regards,

Bernd

P.S.: I like your program, no question.
Even when I don't want to use it.
It's a nice piece of software :!:

Re: Windows Cryptor - wip

Posted: Sat Aug 14, 2010 1:43 pm
by c4s
Nice program, but there seem to be some errors and I also have some ideas:
- Should the "active key character" on the window be transparent?
- Add an option to always hide the task list
- When I encrypt a file with key A and want to decrypt it using B a message appears that A is needed. But When I create another A key that is different from the previous A key I get this message:
"Unable to create output file 843f89j4tj340b0m45b"
- A pause option would be useful
- Double-click on tray icon should open the window as well
- Show/Hide and Create/Remove keyfile should be merged
- Option to toggle always on top (especially for the task window)
- Minimize task window
- When chosing a save path for the keyfile the main window + tasklist can't be moved, could be annoying when those windows are over the path selector

;)

Re: Windows Cryptor - wip

Posted: Sun Aug 15, 2010 11:14 am
by cas
Very nice app 8) . One thing i noticed is that there is ListIcon resize problem in Task List, it resizes beyond window border.

Re: Windows Cryptor - wip

Posted: Sun Aug 15, 2010 7:07 pm
by netmaestro
New update today! Changes include:

- Bug fixed: Files now can be decrypted anywhere (thanks vera)
- Bug fixed: Program can now recognize a wrong key with the right ID (thanks c4s)
- Bug fixed: Tasklist window resizes properly (thanks cas)

Improvements:

- Key installation is more visual
- An option to include subfolders is available on the menu. If checked, a dropped folder and all subfolder contents will be processed
- original files get seven passes of zerobytes instead of one before deletion

Latest version is uploaded, get it at the link in first post.

Re: Windows Cryptor - wip

Posted: Sun Aug 15, 2010 10:15 pm
by netmaestro
Fixed another bug which caused problems when encountering a zero-length file. Handles these properly now.

Re: Windows Cryptor - wip

Posted: Mon Aug 16, 2010 12:13 am
by netmaestro
Sorry one more quick thing: The CRC32 fingerprint of a group of bytes somewhere in the middle of the IV is stored and checked on decryption now, this is to identify a wrong key that has the right ID. This means any .enc files made with the previous version won't decrypt with this one as it will complain that the key is wrong (even if it's right). So best to decrypt or delete all your .enc files before trying this (hopefully last) version.

Now I really am going to move on to something else :mrgreen:

Re: Windows Cryptor - completed

Posted: Mon Aug 16, 2010 3:50 am
by netmaestro
Source is listed in the first post

Re: Windows Cryptor - completed w/source

Posted: Mon Aug 16, 2010 7:01 am
by flaith
:D Thanks a lot Lloyd, you deserve a little dance Image :mrgreen:

Re: Windows Cryptor - completed w/source

Posted: Mon Aug 16, 2010 9:53 am
by infratec
Hi netmaestro,

here are my poor 'optimizations':

1. Combined Hide/Show

replace

Code: Select all

Select EventMenu
        Case 1
          HideWindow(0,0)
         
        Case 2
          HideWindow(0,1)
with

Code: Select all

      Select EventMenu
;        Case 1
;           HideWindow(0,0)
         
        Case 2
          ;          HideWindow(0,1)
          If GetMenuItemText(0, 2) = "Hide"
            HideWindow(0,1)
            SetMenuItemText(0, 2, "Show")
          Else
            HideWindow(0,0)
            SetMenuItemText(0, 2, "Hide")
          EndIf
replace

Code: Select all

CreatePopupMenu(0)
MenuItem(1, "Show")
MenuItem(2, "Hide")
with

Code: Select all

CreatePopupMenu(0)
;MenuItem(1, "Show")
MenuItem(2, "Hide")
2. Allow only 'Show Task List' if not already active

replace

Code: Select all

        Case 5
          HideWindow(2,0)
with

Code: Select all

        Case 5
          HideWindow(2,0)
          DisableMenuItem(0, 5, 1)
replace

Code: Select all

Case #PB_Event_CloseWindow
      HideWindow(2,1)
with

Code: Select all

Case #PB_Event_CloseWindow
      HideWindow(2,1)
      DisableMenuItem(0, 5, 0)
replace

Code: Select all

c$ = EventDropFiles()
          HideWindow(#Window_Tasklist,0)
          If FileSize(c$) = -2
with

Code: Select all

c$ = EventDropFiles()
          HideWindow(#Window_Tasklist,0)
          DisableMenuItem(0, #MenuItem_ShowTasklist, 1)
          If FileSize(c$) = -2
and replace

Code: Select all

HideWindow(#Window_Tasklist,0)
        WriteLog("Too busy to install key, try again later",3)
with

Code: Select all

HideWindow(#Window_Tasklist,0)
        DisableMenuItem(0, 5, 1)
        WriteLog("Too busy to install key, try again later",3)
3. Allow only 'Remove Key' if one is installed

replace

Code: Select all

MenuItem(4, "Remove Key")
MenuItem(5, "Show Task List")
with

Code: Select all

MenuItem(4, "Remove Key")
DisableMenuItem(0, 4, 1)
MenuItem(5, "Show Task List")
replace

Code: Select all

keyinstalled = #True
  currentbase = *baseimage
with

Code: Select all

keyinstalled = #True
  DisableMenuItem(0, 4, 0)
  currentbase = *baseimage
and at least replace

Code: Select all

Case 4
          FillMemory(*keydata, SizeOf(KEYSTRUCT), 0, #PB_Byte)
          keyinstalled=0
          currentbase=*baseimage_d
          RenderBar(0)
with

Code: Select all

Case 4
          FillMemory(*keydata, SizeOf(KEYSTRUCT), 0, #PB_Byte)
          keyinstalled=0
          currentbase=*baseimage_d
          RenderBar(0)
          DisableMenuItem(0, 4, 1)
If it works :oops: , it would be nice if you will use this modifications in future releases. :D

Best regards,

Bernd

P.S.: If you use enumerated constants for menu items, gadgets and so on,
it is much easier to read and find some locations and it is easier to change something.
Yes, it makes a bit more work, but after all you need less time for programming and maintaining the whole program.
Even for 'small' tools this makes sense.

Re: Windows Cryptor - completed w/source

Posted: Thu Aug 19, 2010 1:18 am
by netmaestro
Fixed a couple bugs relating to read-only files, they weren't being processed properly before. Code and exe DL are updated.

Re: Windows Cryptor - completed w/source

Posted: Sat Sep 04, 2010 7:28 am
by loadstone
Very nice ，thanks ：） :D
All times are UTC+01:00
Page 3 of 4

Powered by phpBB® Forum Software © phpBB Limited