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

A BUG in binary encoding storage conversion #49883

Closed
VectorWorkshopBaoErJie opened this issue Jun 24, 2021 · 2 comments
Closed

A BUG in binary encoding storage conversion #49883

VectorWorkshopBaoErJie opened this issue Jun 24, 2021 · 2 comments

Comments

@VectorWorkshopBaoErJie
Copy link

VectorWorkshopBaoErJie commented Jun 24, 2021

Godot version

3.3.stable+

System information

Windows10

Issue description

project_settings.cpp :A BUG in binary encoding storage conversion

Steps to reproduce

    //file:project_settings.cpp    function name:_save_settings_binary
    if (p_custom_features != String()) {
    file->store_32(count + 1);
    //store how many properties are saved, add one for custom featuers, which must always go first
    String key = CoreStringNames::get_singleton()->_custom_features;
    file->store_32(key.utf8().length()); //Fixed BUG location
    file->store_string(key);
    int len;
    err = encode_variant(p_custom_features, NULL, len, false);
    if (err != OK) {
        memdelete(file);
        ERR_FAIL_V(err);
 }

Minimal reproduction project

No response

@Calinou
Copy link
Member

Calinou commented Jun 24, 2021

@qjyqjyqjyqjy Can you describe the incorrect behavior and what the proposed code fixes?

@timothyqiu
Copy link
Member

timothyqiu commented Jun 24, 2021

This has been fixed by #49649, I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants