Wrong returned value in the library function SD_initialize()?

[JSmit.18]

I am having problem with FATFS remounting with Cube IDE and STM327H50.  Tracing the program surfaced the SD_initialize() function which is defined in the ST library file  FATFS/Target/sd_diskio.c. 

The problem is that when BSP_SD_Init() failed, SD_initialize() returns whatever value previous functions left in the variable Stat, which can cause SD_initialize() to sometimes return OK even if BSP_SD_Init() failed.

Am I missing something?

 

 

/**
 * @brief Initializes a Drive
 * @PAram lun : not used
 * @retval DSTATUS: Operation status
 */
DSTATUS SD_initialize(BYTE lun)
{
#if !defined(DISABLE_SD_INIT)
 if(BSP_SD_Init() == MSD_OK)
 {
 Stat = SD_CheckStatus(lun);
 }
#else
 Stat = SD_CheckStatus(lun);
#endif
 return Stat;
}