bcox/WaveTools
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

check for the correct padding, INFO #1

Merged
ryanfrancesconi merged 1 commit from master into master 2017-06-03 04:12:58 +00:00
Conversation 1 Commits 1 Files changed 1 +7
ryanfrancesconi commented 2017-06-02 17:41:05 +00:00 (Migrated from github.com)
Copy link

I'm not sure if you're still maintaining this framework, but it has been a big help to me. Wanted to submit this small fix. It was writing odd length INFO chunks which is against the wave spec and crashes some apps. Thanks! Ryan

An odd byte length chunks exists that is not immediately followed by a NULL byte.
This could cause problems in reading subsequent chunks in some systems so add one more byte.

I'm not sure if you're still maintaining this framework, but it has been a big help to me. Wanted to submit this small fix. It was writing odd length INFO chunks which is against the wave spec and crashes some apps. Thanks! Ryan An odd byte length chunks exists that is not immediately followed by a NULL byte. This could cause problems in reading subsequent chunks in some systems so add one more byte.
djehuti commented 2017-06-03 04:13:07 +00:00 (Migrated from github.com)
Copy link

Thanks!

Thanks!
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
bcox/WaveTools!1
No description provided.