diff options
| author | William Wilgus <wilgus.william@gmail.com> | 2024-06-02 22:14:27 -0400 |
|---|---|---|
| committer | William Wilgus <wilgus.william@gmail.com> | 2024-06-02 22:14:27 -0400 |
| commit | afba86f0f79d314f0414975fa71db6a6a41118cb (patch) | |
| tree | 12199c8faa24fe06a03764496559c2f2693e32da /apps | |
| parent | 283f31dd399d839d597fa48ae98dcdccb6188fd7 (diff) | |
| download | rockbox-afba86f0f79d314f0414975fa71db6a6a41118cb.tar.gz rockbox-afba86f0f79d314f0414975fa71db6a6a41118cb.zip | |
[coverity] playlist.c pl_save_update_control() close fd on error
Change-Id: Ib44e86a36540e65a15eed3e9b99106734379250d
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/playlist.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/apps/playlist.c b/apps/playlist.c index 9935643f5d..7040c2a129 100644 --- a/apps/playlist.c +++ b/apps/playlist.c | |||
| @@ -3931,7 +3931,10 @@ static int pl_save_update_control(struct playlist_info* playlist, | |||
| 3931 | playlist->filename[playlist->dirlen-1] = c; | 3931 | playlist->filename[playlist->dirlen-1] = c; |
| 3932 | 3932 | ||
| 3933 | if (err <= 0) | 3933 | if (err <= 0) |
| 3934 | { | ||
| 3935 | close(old_fd); | ||
| 3934 | return -4; | 3936 | return -4; |
| 3937 | } | ||
| 3935 | 3938 | ||
| 3936 | if (playlist->first_index > 0) | 3939 | if (playlist->first_index > 0) |
| 3937 | { | 3940 | { |