Storage object creation doesn't follow the specification

[Sonic-The-Hedgehog-LNK1123]

See the below quote from the specification:

If an implementation of the file format enables applications to create
storage objects without explicitly setting an object class GUID, it MUST write all zeroes by default.
If this value is not all zeroes, the object class GUID can be used as a parameter to start
applications.

To fix this, change:

openmcdf/sources/OpenMcdf/DirectoryEntry.cs

Line 190 in 04b53dc

= Guid.NewGuid();

To:

 = Guid.Empty;

This change makes the switch block:

openmcdf/sources/OpenMcdf/DirectoryEntry.cs

Line 54 in 04b53dc

switch (stgType)

even more redundant than it already was, you're setting member variables to zero that you're already initialized to zero at their declaration.
Also, this:

openmcdf/sources/OpenMcdf/DirectoryEntry.cs

Line 58 in 04b53dc

this.storageCLSID = new Guid("00000000000000000000000000000000");

is one of the most inefficient ways to set a GUID to Guid.Empty.

Replacing that switch block with this if statement:

if (stgType == StgType.StgStorage)
{
    this.creationDate = BitConverter.GetBytes((DateTime.Now.ToFileTime()));
}

will have the same effect as that switch block.

[ironfede]

Thank you for your advices.
Best Regards,
Federico