Remove malloc call from gps.c

[Derek Chan]

Only noticed this after doing all the merging settled. This issue is just as a placeholder so we don't forget to make the change.

We should remove malloc calls and instead have the caller function provide a reference instead (as indicated in your comment). I agree with you that be better for performance and that we don't want to deal with memory leaks.

Code ref: https://github.com/dchanman/cpen391-21/blob/master/minecraft_go/software/mc_go/src/gps.c#L40