[apparmor] [patch] regression tests: fix gcc-5 inline confusion

[Seth Arnold]

On Tue, Mar 03, 2015 at 10:58:57AM -0800, Steve Beattie wrote:
> Parts of the regression tests that use the do_open() inline function
> from changehat.h fail to build under gcc-5 like so:
> 
>   cc -g -O0 -Wall -Wstrict-prototypes    changeprofile.c  -lapparmor  -o changeprofile
>   /tmp/ccT6GE6k.o: In function `main':
>   /home/ubuntu/bzr/apparmor/tests/regression/apparmor/changeprofile.c:43: undefined reference to `do_open'
>   collect2: error: ld returned 1 exit status
>   <builtin>: recipe for target 'changeprofile' failed
> 
> This patch converts the do_open function declaration to be static
> inline, which apparently keeps gcc-5 from getting confused.
> 
> Signed-off-by: Steve Beattie <steve at nxnw.org>

Acked-by: Seth Arnold <seth.arnold at canonical.com>

Thanks

> ---
>  tests/regression/apparmor/changehat.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: b/tests/regression/apparmor/changehat.h
> ===================================================================
> --- a/tests/regression/apparmor/changehat.h
> +++ b/tests/regression/apparmor/changehat.h
> @@ -4,7 +4,7 @@
>  
>  #define SD_ID_MAGIC     0x8c235e38
>  
> -inline int do_open (char * file)
> +static inline int do_open (char * file)
>  {
>  	int fd, rc;
>  	char buf[128];
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20150303/322a85f2/attachment.pgp>