[CVS freeldr] Report on which drive we couldn't find a partition table

[Casper Hornstrup]

Shouldn't ErrMsg be passed to DiskError()?

Casper

_____

From: [e-mail] [mailto:[e-mail]] On Behalf Of Ge van Geldorp
Sent: 21. november 2004 13:28
To: [e-mail]
Subject: [ros-diffs] [CVS freeldr] Report on which drive we couldn't find a partition table

Commit in freeldr/freeldr/disk on MAIN	
partition.c <> 	 +3	 	1.12
http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk/partition.c?rev=1.12&content-type=text/x-cvsweb-markup  -
http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk/partition.c.diff?r1=text&tr1=1.12&r2=text&tr2=1.13&f=h > 1.13
http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk/partition.c?rev=1.13&content-type=text/x-cvsweb-markup 	
Report on which drive we couldn't find a partition table
  _____

freeldr http://cvs.reactos.com/cgi-bin/cvsweb/freeldr /freeldr http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr /disk
http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk

partition.c 1.12 http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk/partition.c?rev=1.12&content-type=text/x-cvsweb-markup
- http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk/partition.c.diff?r1=text&tr1=1.12&r2=text&tr2=1.13&f=h > 1.13
http://cvs.reactos.com/cgi-bin/cvsweb/freeldr/freeldr/disk/partition.c?rev=1.13&content-type=text/x-cvsweb-markup 
diff -u -r1.12 -r1.13

--- partition.c	9 Nov 2004 23:36:20 -0000	1.12

+++ partition.c	21 Nov 2004 12:28:12 -0000	1.13

@@ -197,6 +197,7 @@

BOOL DiskReadBootRecord(U32 DriveNumber, U64 LogicalSectorNumber, PMASTER_BOOT_RECORD BootRecord)

{
+	char		ErrMsg[64];
 #ifdef DEBUG

U32		Index;

#endif
@@ -236,6 +237,8 @@

// Check the partition table magic value

if (BootRecord->MasterBootRecordMagic != 0xaa55)

{
+		sprintf(ErrMsg, "Invalid partition table magic 0x%x found on drive 0x%x",

+		        BootRecord->MasterBootRecordMagic, DriveNumber);
    	DiskError("Invalid partition table magic (0xaa55)", 0);

return FALSE;

}
CVSspam http://www.badgers-in-foil.co.uk/projects/cvsspam/  0.2.8

[Ge van Geldorp]

> From: Casper Hornstrup
>
> Shouldn't ErrMsg be passed to DiskError()?

Doh...
/me looks sheepish
Thanks for catching that.

Ge van Geldorp