Fix GH-20076: open_basedir prohibits opening :memory:?cache=shared #20080
Conversation
|
Fails for some reason, marking as draft until I figured it out. |
| return estrdup(filename); | ||
| } | ||
| if (*filename && strcmp(filename, ":memory:")) { | ||
| if (*filename && strncmp(filename, ":memory:", sizeof(":memory:")-1) != 0) { |
There was a problem hiding this comment.
Is this 100% safe and wouldn't be something like :memory:/path processed like regular filepath?
There was a problem hiding this comment.
It seems this is not only theoretical, but also the real reason why the CI is failing - a file (instead of in-memory DB) is created.
There was a problem hiding this comment.
The official condition is in https://github.com/sqlite/sqlite/blob/version-3.50.4/src/btree.c#L2552.
I even wonder what is :memory:?cache=shared, I did not find any testcase for it in Sqlite codebase.
There was a problem hiding this comment.
The "cache" connection parameter lets you specify the "shared key" to connect to in-memory database created in a different process.
Ref: https://www.sqlite.org/inmemorydb.html#sharedmemdb
The value could be anything, on production we're using UUID generated at deploy time.
No description provided.