Check return codes everywhere

[elfring]

Would you like to add more error handling for return values from functions like the following?

• ftell ⇒ read_file
• malloc ⇒ mypaint_brush_new

[achadwick]

Sounds reasonable. Those functions should probably return NULL if the functions you've identified fail.

Do you have a patch, and/or can you further define the scope for this? "Everywhere" is too broad to work from.

[elfring]

I suggest to avoid ignorance of return values a bit more. How do you think about to improve static source code analysis also for this software?

[achadwick]

We generally like to automate tests in response to github commits. Perhaps point coverity at it? https://scan.coverity.com/travis_ci