keremet/rpm-ostree
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

build: Use -fvisibility=hidden and explicit exports

Browse Source 
This avoids symbol leakage.  Previously we were at risk of exporting
other random API from libpriv, as well as libglnx.
This commit is contained in:
Colin Walters 2015-04-13 21:25:05 -04:00
parent de47c23450
commit 792da457db
4 changed files with 17 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Makefile-lib.am
View File

@ -29,8 +29,9 @@ librpmostree_1_la_SOURCES = \
$(NULL) $(NULL)
librpmostree_1_la_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/libglnx -I$(srcdir)/src/libpriv -I$(srcdir)/src/lib \ librpmostree_1_la_CFLAGS = $(AM_CFLAGS) -I$(srcdir)/libglnx -I$(srcdir)/src/libpriv -I$(srcdir)/src/lib \
-fvisibility=hidden '-D_RPMOSTREE_EXTERN=__attribute((visibility("default"))) extern' \
$(PKGDEP_RPMOSTREE_CFLAGS) $(PKGDEP_RPMOSTREE_CFLAGS)
librpmostree_1_la_LDFLAGS = -version-number 1:0:0 -Bsymbolic-functions -export-symbols-regex '^rpm_ostree_' librpmostree_1_la_LDFLAGS = -version-number 1:0:0 -Bsymbolic-functions
librpmostree_1_la_LIBADD = libglnx.la $(PKGDEP_RPMOSTREE_LIBS) librpmostree_1_la_LIBADD = libglnx.la $(PKGDEP_RPMOSTREE_LIBS)
if BUILDOPT_INTROSPECTION if BUILDOPT_INTROSPECTION

3
Makefile.am
View File

@ -36,7 +36,8 @@ AM_CFLAGS += $(WARN_CFLAGS)
EXTRA_DIST += autogen.sh COPYING EXTRA_DIST += autogen.sh COPYING
libglnx_srcpath := $(srcdir)/libglnx libglnx_srcpath := $(srcdir)/libglnx
libglnx_cflags := $(PKGDEP_GIO_UNIX_CFLAGS) -I$(libglnx_srcpath) libglnx_cflags := $(PKGDEP_GIO_UNIX_CFLAGS) -I$(libglnx_srcpath) -fvisibility=hidden
libglnx_ldflags :=
libglnx_libs := $(PKGDEP_GIO_UNIX_LIBS) libglnx_libs := $(PKGDEP_GIO_UNIX_LIBS)
include $(INTROSPECTION_MAKEFILE) include $(INTROSPECTION_MAKEFILE)

10
src/lib/rpmostree-db.h
View File

@ -26,13 +26,13 @@ G_BEGIN_DECLS
typedef struct RpmOstreeDbQueryResult RpmOstreeDbQueryResult; typedef struct RpmOstreeDbQueryResult RpmOstreeDbQueryResult;
GType rpm_ostree_db_query_result_get_type (void); _RPMOSTREE_EXTERN GType rpm_ostree_db_query_result_get_type (void);
const char *const *rpm_ostree_db_query_result_get_packages (RpmOstreeDbQueryResult *queryresult); _RPMOSTREE_EXTERN const char *const *rpm_ostree_db_query_result_get_packages (RpmOstreeDbQueryResult *queryresult);
RpmOstreeDbQueryResult *rpm_ostree_db_query_ref (RpmOstreeDbQueryResult *result); _RPMOSTREE_EXTERN RpmOstreeDbQueryResult *rpm_ostree_db_query_ref (RpmOstreeDbQueryResult *result);
void rpm_ostree_db_query_unref (RpmOstreeDbQueryResult *result); _RPMOSTREE_EXTERN void rpm_ostree_db_query_unref (RpmOstreeDbQueryResult *result);
RpmOstreeDbQueryResult *rpm_ostree_db_query (OstreeRepo *repo, _RPMOSTREE_EXTERN RpmOstreeDbQueryResult *rpm_ostree_db_query (OstreeRepo *repo,
const char *ref, const char *ref,
GVariant *query, GVariant *query,
GCancellable *cancellable, GCancellable *cancellable,

4
src/lib/rpmostree.h
View File

@ -20,4 +20,8 @@
#pragma once #pragma once
#ifndef _RPMOSTREE_EXTERN
#define _RPMOSTREE_EXTERN extern
#endif
#include <rpmostree-db.h> #include <rpmostree-db.h>