[GUACAMOLE-482] Return value of guacenc_video_flush_frame() not handled

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 0.9.13-incubating
• Fix Version/s: 1.0.0
• Component/s: guacenc
• Environment: using provided dockerfile build
• Assignee: Mike Jumper
• Reporter: Ryan Underwood
• Created: 12/Jan/18 20:17
• Updated: 17/Apr/18 19:35
• Resolved: 17/Apr/18 19:35

Description

In this file: https://github.com/apache/guacamole-server/blob/master/src/guacenc/display-sync.c line 52, the function called "guacenc_video_advance_timeline" doesn't appear to have a path that can return 1.
Reporting because I wasn't sure if this was an artifact from a refactor, a bug where guacenc_video_advance_timeline should have a path that returns 1, or if it was just a bit of extra code that can go away.