[Bug 207134] graphics/libfpx: fix build with clang 3.8.0

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=207134

            Bug ID: 207134
           Summary: graphics/libfpx: fix build with clang 3.8.0
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Some People
          Priority: ---
         Component: Individual Port(s)
          Assignee: freebsd-ports-bugs at FreeBSD.org
          Reporter: dim at FreeBSD.org
                CC: mi at ALDAN.algebra.com
                CC: mi at ALDAN.algebra.com
             Flags: maintainer-feedback?(mi at ALDAN.algebra.com)

Created attachment 166903
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=166903&action=edit
Fix infinite loop in libfpx's EB_Copy_To_Memory() function

During a recent exp-run for bug 206074, it was found that graphics/libfpx gives
errors with clang 3.8.0:

/wrkdirs/usr/ports/graphics/libfpx/work/libfpx-1.3.1-4/jpeg/ebuffer.c:94:36:
error: variables 'i' and 'num' used in loop condition not modified in loop body
[-Werror,-Wfor-loop-analysis]
  for(i=0,mem_ptr=mem,buf_ptr=buf; i < num ;*mem_ptr++=*buf_ptr++);
                                   ^   ~~~

This for loop, in the EB_Copy_To_Memory() function, is non-terminating, since
it never increases its counter.  The attached patch fixes this.

[1]
http://package22.nyi.freebsd.org/data/headamd64PR206074-default/2016-02-12_13h52m56s/logs/errors/libfpx-1.3.1.4.log

-- 
You are receiving this mail because:
You are the assignee for the bug.