Skip to content
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.

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

Fix out of boundary access in src_linear.c #209

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lkoenig
Copy link

@lkoenig lkoenig commented Mar 6, 2024

Add a test case with different consecutive buffer size and a fix for #208.

Fix libsndfile#208 about out of bount access in linear resampler.

Also migrates to count the sumber of frames instead of the number of
samples and remove some redundant member of the private state.

Signed-off-by: Lionel Koenig Gélas <lionel.koenig@gmail.com>
@lkoenig lkoenig changed the title Test case to catch out of bundary access in src_linear.c Fix out of boundary access in src_linear.c Mar 7, 2024
@lkoenig
Copy link
Author

lkoenig commented Mar 7, 2024

@erikd and @evpobr would you mind having a look at this PR ?

@evpobr evpobr self-assigned this May 11, 2024
@evpobr evpobr added this to the v0.2.3 milestone May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants