Serial Strangeness

25 posts

angusiNovember 21, 2007 at 7:05 PM#

Serial Strangeness

Hi there 🙂

I have this code which i use to validate a serial code: the user types their serial code in a box which then posts to a webpage to check against a MySQL database and says VALID if the key is valid and INVALID if not

if the key is entered correctly, there is no problem. If the key is NOT then the page for entering the serial is shown again. The only difference is that when you enter the key the second time, it doesn't matter if the key is correct or not, it doesn't bother to validate. Please advise. I have posted the bit of code which validates below:



Function SerialPageShow

	!insertmacro MUI_HEADER_TEXT "Enter Serial Code" "Enter the software serial code to continue."



	PassDialog:😁ialog Serial            \

                    /HEADINGTEXT 'Please enter the serial code...' \

                    /CENTER             \

                    /BOXDASH 12  70 5 '' \

                    /BOXDASH 92  70 5 ''  \

                    /BOXDASH 172 70 5 ''   \

                    /BOXDASH 252 70 5 ''    \

                    /BOX     332 70 5 ''



Pop $R0 # success, back, cancel or error

FunctionEnd



Function SerialPageLeave



	## Pop values from stack

	Pop $R1

	Pop $R2

	Pop $R3

	Pop $R4

	Pop $R5



	;!insertmacro MUI_HEADER_TEXT "Validating..." "Please Wait"

	;!insertmacro MUI_INSTALLOPTIONS_DISPLAY "validateSerial.ini"



	;MessageBox MB_OK "Break1"



FunctionEnd



Function validateSerial



	;MessageBox MB_OK "Break2"



	;## Disable the Back button

	;GetDlgItem $R6 $HWNDPARENT 3

	;EnableWindow $R6 0

	;## Disable the Next button

	;GetDlgItem $R6 $HWNDPARENT 1

	;EnableWindow $R6 0



	GetTempFileName $0

	InetLoad::load /post "licence=$R1-$R2-$R3-$R4-$R5" "http://www.mysite.co.uk/install/verify.php" "$0" /slient ""

	Pop $1 # $1 now holds the exit code for above



	FileOpen $2 $0 "r"

	FileRead $2 $3

	FileClose $2



	${If} $3 == 'INVALID'

	MessageBox MB_OK|MB_ICONEXCLAMATION "Invalid Serial"

	Call SerialPageShow

	${Else}

	Goto serialOK

	${EndIf}



	Abort



	serialOK:

FunctionEnd

Afrow UKNovember 21, 2007 at 7:58 PM#

Move the contents of validateSerial into SerialPageLeave.
You don't Call SerialPageShow to show the page again... you just call Abort.

Stu

angusiNovember 22, 2007 at 5:05 PM#

you mean like this:



Function SerialPageLeave



	## Pop values from stack

	Pop $R1

	Pop $R2

	Pop $R3

	Pop $R4

	Pop $R5



	;!insertmacro MUI_HEADER_TEXT "Validating..." "Please Wait"

	;!insertmacro MUI_INSTALLOPTIONS_DISPLAY "validateSerial.ini"



	;MessageBox MB_OK "Break1"





	;MessageBox MB_OK "Break2"



	;## Disable the Back button

	;GetDlgItem $R6 $HWNDPARENT 3

	;EnableWindow $R6 0

	;## Disable the Next button

	;GetDlgItem $R6 $HWNDPARENT 1

	;EnableWindow $R6 0



	GetTempFileName $0

	InetLoad::load /post "licence=$R1-$R2-$R3-$R4-$R5" "http://www.mysite.co.uk/site/verify.php" "$0" /slient ""

	Pop $1 # $1 now holds the exit code for above



	FileOpen $2 $0 "r"

	FileRead $2 $3

	FileClose $2



	${If} $3 == 'INVALID'

	MessageBox MB_OK|MB_ICONEXCLAMATION "Invalid Serial"

	Call SerialPageShow

	${Else}

	Goto serialOK

	${EndIf}



	Abort



	serialOK:



FunctionEnd

or like this:



Function SerialPageLeave

	;MessageBox MB_OK "Break1"





	;MessageBox MB_OK "Break2"



	;## Disable the Back button

	;GetDlgItem $R6 $HWNDPARENT 3

	;EnableWindow $R6 0

	;## Disable the Next button

	;GetDlgItem $R6 $HWNDPARENT 1

	;EnableWindow $R6 0



	GetTempFileName $0

	InetLoad::load /post "licence=$R1-$R2-$R3-$R4-$R5" "http://www.mysite.co.uk/site/verify.php" "$0" /slient ""

	Pop $1 # $1 now holds the exit code for above



	FileOpen $2 $0 "r"

	FileRead $2 $3

	FileClose $2



	${If} $3 == 'INVALID'

	MessageBox MB_OK|MB_ICONEXCLAMATION "Invalid Serial"

	Call SerialPageShow

	${Else}

	Goto serialOK

	${EndIf}



	Abort



	serialOK:



FunctionEnd

Using the first method, the installer exits the second time a code is entered whether valid or not.

Afrow UKNovember 22, 2007 at 6:50 PM#

Function SerialPageLeave



  ## Pop values from stack

  Pop $R1

  Pop $R2

  Pop $R3

  Pop $R4

  Pop $R5



  GetTempFileName $0

  InetLoad::load /post "licence=$R1-$R2-$R3-$R4-$R5" /silent "" "http://www.mysite.co.uk/site/verify.php" "$0" /end

  Pop $1 # $1 now holds the exit code for above



  FileOpen $2 $0 "r"

  FileRead $2 $3

  FileClose $2



  ${If} $3 == 'INVALID'

    MessageBox MB_OK|MB_ICONEXCLAMATION "Invalid Serial"

    Abort

  ${EndIf}



FunctionEnd

Notice I have changed your InetLoad parameters a bit so that it's to the readme spec.

Stu

angusiNovember 23, 2007 at 1:37 PM#

Edit (again!): The installer now simply says the code is invalid, even if the database doesn't say INVALID, but says Serial Validated

Afrow UKNovember 23, 2007 at 5:03 PM#

I have swapped the /silent and /post switched around.

Stu

angusiNovember 24, 2007 at 10:56 AM#

so should I switch them round?
still says invalid for all serials

don't know if this is just my pc, but the installer takes at least 20 seconds to try and check (and fail) the serial the first time then only about 2 seconds the second, third, fourth etc times.

Afrow UKNovember 24, 2007 at 12:09 PM#

Check the exit code. Your code would not have worked because all switches go before the url parts, not to mention you had /slient not /silent 🙂

Stu

angusiNovember 24, 2007 at 12:29 PM#

ok... i'm now a little confused... can you post the code to show what you mean... thanks.

Afrow UKNovember 24, 2007 at 6:57 PM#

Pop $1 # $1 now holds the exit code for above

Check the value of $1.

Stu

angusiNovember 24, 2007 at 7:01 PM#

umm... how would i do that?

Afrow UKNovember 24, 2007 at 7:42 PM#

With a MessageBox perhaps 🙂

Stu

angusiNovember 25, 2007 at 9:02 AM#

ahh 🙂

like this: MessageBox MB_OK|MB_ICONEXCLAMATION "$1"

the message box says "SendRequest Error"

and the installer has now started validating every code... 😕

if you want to take a look at the installer to see for yourself i can post it here.

Afrow UKNovember 25, 2007 at 1:17 PM#

If there is an error then obviously there is something wrong. Try removing the /silent "" bit.

Stu

angusiNovember 25, 2007 at 2:28 PM#

the installer just goes straight onto the components page and doesn't try to access the validate.php file

Afrow UKNovember 25, 2007 at 3:23 PM#

There must be something wrong with your php script or website. The code works fine for me using a different php script with post data.

Stu

TakhirNovember 25, 2007 at 3:36 PM#

Please note, that FileRead returns string with newline char at the end (if any), so comparision may require a kind of trim for incoming string (or just use 'INVALIDE$\n' in ${if} statement). The following test works correct for me:

    InetLoad::load /post "licence=INVALID" /silent "" "http://localhost/echo.php" "$EXEDIR\echo.htm" /end

Pop $0 # "OK" means okay

FileOpen $2 "$EXEDIR\echo.htm" "r"

FileRead $2 $3

FileClose $2

MessageBox MB_OK "Download Status: $0, Result = <$3>"

StrCmp $3 "INVALID" 0 +2

MessageBox MB_OK "Invalid Serial"

for the following test php

<?php $n = 0; foreach($_REQUEST as $v) {echo $v; if($n > 0) echo "\n"; $n++; } ?>

But comparision require 'INVALIDE$\n' if newline presents in the server reply.

angusiNovember 26, 2007 at 5:11 PM#

Originally posted by Afrow UK
There must be something wrong with your php script or website. The code works fine for me using a different php script with post data.

Stu

what is your php script like?
i've checked the php manually (launched my browser and gone to the URL) and when i "post" an invalid code the source of the page simply says

INVALID

and when i "post" a valid one,

Serial Validated

Thats all the source shows.

Afrow UKNovember 26, 2007 at 5:46 PM#

My PHP script is just a normal webpage. The only thing I can think of is make sure you set the content type to plain text before displaying your result in the PHP script:

header("Content-Type: text/plain");

Stu

angusiNovember 29, 2007 at 1:06 PM#

:l still doesn't work...

a temporary URL for the script is Here - At the moment any code should read INVALID...

angusiDecember 1, 2007 at 1:49 PM#

This is really starting to annoy me now lol... I tried putting some of this into my script:

Originally posted by Takhir



InetLoad::load /post "licence=$R1-$R2-$R3-$R4-$R5" /silent "" "http://www.mysite.com/site/verify.php" "$EXEDIR\echo.htm" /end

Pop $0 # "OK" means okay

FileOpen $2 "$EXEDIR\echo.htm" "r"

FileRead $2 $3

FileClose $2

MessageBox MB_OK "Download Status: $0, Result = <$3>"

StrCmp $3 "INVALID" 0 +2

MessageBox MB_OK "Invalid Serial"

the $EXEDIR\echo.htm reads INVALID, but not in text/plain formatting - is that somehow relevant?

Afrow UKDecember 2, 2007 at 12:35 PM#

Make sure you aren't placing any new line characters on the end of the output string. FileRead reads new line characters as well.

Stu

angusiDecember 2, 2007 at 1:20 PM#

would that not mean that if would validate every code?

Afrow UKDecember 2, 2007 at 2:15 PM#

Sorry?
Just make sure in your PHP script you're echoing "INVALID" and not "INVALID\r\n" for example.

What does your message box display?
MessageBox MB_OK "Download Status: $0, Result = <$3>"

Stu

angusiDecember 4, 2007 at 4:45 PM#

"INVALID"