Journal: copy to Documents crashes Sugar

[manuq]

• create a screenshot with Alt-1
• go to Journal
• open the palette of the screenshot item
• select Copy To -> Documents

Crash.

[tch]

I have isolated this problem to

src/jarabe/journal/model.py
"SugarExt.fat_set_hidden_attrib" call

I see two options:

1. I check if path_file is in Documents folder, so I don't call this function.
2. I modify this function to make sure the file belongs a fat fs.

Any suggestions?

[godiard]

+1 to option a

[tch]

Ok, going deeper, found that the crash is caused by something at

sugar-toolkit-gtk3/src/sugar3/sugar-fatattr.c
"fprintf(stdout, "Error %s attributes: %s\n", verb, strerror(errno));"

errno returns 25, but strerror(25) returns garbage string...

While I was reading about this, saw that strerror is not thread-safe, and switched to strerror_r. This stops sugar from crashing, but does not solve the garbage string issue.

[tch]

Dsd gave a brief talk about including string.h when using strerror, which solves this issues. Just sent a pull-request.

[Martin Abente Lahaye]

Include string.h to sugar-fatattr

According to dsd's wisdom, without being
included, strerror is undeclared.

Fixes #4600 sugar crashing when copying
journal entries to documents folder.

Signed-off-by: Martin Abente Lahaye <tch@…>

Changeset: ba8c435e5091c601820d6d08e7def119b0c8d3c3