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

[Casper Hornstrup]

Shouldn't ErrMsg be passed to DiskError()?
 
Casper


  _____  

From: ros-diffs-bounces(a)reactos.com [mailto:[e-mail]] On Behalf Of Ge
van Geldorp
Sent: 21. november 2004 13:28
To: ros-diffs(a)reactos.com
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