Skip to content

FileSystem: correct fchdrive to correctly handle NULL argument #171

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
VladimirUmek wants to merge 2 commits into
base: main
Choose a base branch
from
Open

FileSystem: correct fchdrive to correctly handle NULL argument #171

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a9548b8
FileSystem: correct fchdrive to correctly handle NULL argument
VladimirUmek Aug 27, 2026
30b68c2
Bump version
VladimirUmek Aug 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Components/FileSystem/FileSystem.scvd
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="utf-8"?>

<component_viewer schemaVersion="0.1" xmlns:xs="http://www.w3.org/2001/XMLSchema-instance" xs:noNamespaceSchemaLocation="Component_Viewer.xsd">
<component name="File System" shortname="FS" version="8.0.6"/> <!-- name and version of the component -->
<component name="File System" shortname="FS" version="8.0.7"/> <!-- name and version of the component -->

<typedefs>
<!-- Flash Sector information (Driver_Flash.h line 58) -->
Expand Down
2 changes: 1 addition & 1 deletion Components/FileSystem/Include/rl_fs.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@

#define MW_FS_VERSION_MAJOR 8
#define MW_FS_VERSION_MINOR 0
#define MW_FS_VERSION_PATCH 6
#define MW_FS_VERSION_PATCH 7

// ==== Enumeration, structures, defines ====

Expand Down
22 changes: 10 additions & 12 deletions Components/FileSystem/Source/fs_mapi.c
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
/*------------------------------------------------------------------------------
* MDK Middleware - Component ::File System
* Copyright (c) 2004-2025 Arm Limited (or its affiliates). All rights reserved.
* Copyright (c) 2004-2026 Arm Limited (or its affiliates). All rights reserved.
*------------------------------------------------------------------------------
* Name: fs_mapi.c
* Purpose: File Maintenance API Functions
Expand Down Expand Up @@ -987,21 +987,19 @@ fsStatus fchdir (const char *path) {
*/
fsStatus fchdrive (const char *drive) {
int32_t id;
fsStatus status;

EvrFsCore_fchdrive (drive);

if (drive[0] == '\0') {
/* Empty string is not allowed */
return (fsInvalidParameter);
}
else {
id = fs_drive_id (drive, NULL);
if (id < 0) {
/* Nonexistent drive or invalid input */
return ((fsStatus)(-id));
}
id = fs_drive_id (drive, NULL);
if (id < 0) {
/* Nonexistent drive or invalid input */
status = (fsStatus)(-id);
} else {
status = fs_set_cdrive (id);
}
return (fs_set_cdrive (id));

return status;
}


Expand Down
3 changes: 2 additions & 1 deletion Documentation/Doxygen/FileSystem/src/revision_history.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,9 @@
<th>Description</th>
</tr>
<tr>
<td>V8.0.6</td>
<td>V8.0.7</td>
<td>
- corrected fchdrive to correctly handle NULL argument
- corrected funmount for FAT drives when called multiple times
- corrected FAT name cache entry size from 48 to 40 bytes
- corrected various minor code quality issues
Expand Down
2 changes: 1 addition & 1 deletion Documentation/Doxygen/General/src/revision_history.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
<td>V8.3.1</td>
<td>
- Network Component Version 8.3.1
- FileSystem Component Version 8.0.6
- FileSystem Component Version 8.0.7
- USB Component Version 8.0.2 (unchanged)
</td>
</tr>
Expand Down
5 changes: 3 additions & 2 deletions Keil.MDK-Middleware.pdsc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@
<releases>
<release version="8.3.1-dev">
Active development ...
FileSystem Component Version 8.0.6
FileSystem Component Version 8.0.7
- corrected fchdrive to correctly handle NULL argument
- corrected funmount for FAT drives when called multiple times
- corrected FAT name cache entry size from 48 to 40 bytes
- corrected various minor code quality issues
Expand Down Expand Up @@ -1164,7 +1165,7 @@
</bundle>

<!-- File System (MDK) -->
<bundle Cbundle="MDK" Cclass="File System" Cversion="8.0.6">
<bundle Cbundle="MDK" Cclass="File System" Cversion="8.0.7">
<description>File Access on various storage devices</description>
<doc>Documentation/html/FileSystem/index.html</doc>
<component Cgroup="CORE" condition="CMSIS Core with RTOS and File System I/O">
Expand Down
Loading