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

SNOW-983100: libsnowflakeclient - banned functions in ODBC v3.1.2 #657

Closed
wants to merge 1 commit into from

Conversation

sfc-gh-ext-simba-jz
Copy link
Collaborator

  1. replace strerror with strerror_r on Posix (with strerror_s on Windows already)
  2. enhance getenv
    2.1. thread safe in driver
    2.2. allow reentrance with cloned value returned
  3. strcpy/strncpy/strcat/strncat, implement bounds check according to spec of "secured" version
  4. related updates
    4.1. consistent API of sf_getenv and sf_strerror on all platforms
    4.2. fix some memory leaks
    4.3. relevant unit tests updates

1. replace strerror with strerror_r on Posix (with strerror_s on Windows already)
2. enhance getenv
   2.1. thread safe in driver
   2.2. allow reentrance with cloned value returned
3. strcpy/strncpy/strcat/strncat, implement bounds check according to spec of "secured" version
4. related updates
   4.1. consistent API of sf_getenv and sf_strerror on all platforms
   4.2. fix some memory leaks
   4.3. relevant unit tests updates
@sfc-gh-ext-simba-jz sfc-gh-ext-simba-jz requested review from a user and sfc-gh-igarish January 18, 2024 00:10
@ghost
Copy link

ghost commented Feb 6, 2024

New PR is made #659
This one should be dropped. @sfc-gh-ext-simba-jz

@sfc-gh-dprzybysz
Copy link
Collaborator

If there is a newer PR we can close this draft

@github-actions github-actions bot locked and limited conversation to collaborators Feb 7, 2024
@sfc-gh-dprzybysz sfc-gh-dprzybysz deleted the b782-msbanned branch May 27, 2024 11:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants