diff options
| author | Niels de Vos <ndevos@redhat.com> | 2016-10-17 11:58:30 +0200 |
|---|---|---|
| committer | Kaleb KEITHLEY <kkeithle@redhat.com> | 2016-11-22 18:03:54 -0800 |
| commit | 76f5061e1fe3c4602ab4f2c935c061a96e60825c (patch) | |
| tree | e85920670c9ba88e5b5f587fb8bfa70121906025 /api | |
| parent | deaea10a63da5ca8ce083a75e74d57b8e41d4139 (diff) | |
gfapi: add glfs_free() to glfs.h
Commit 4721188a154acd9a0a4c096d8d73e97f3bf1b2a9 introduces glfs_free()
but missed adding the function to the header. The symbol is correctly
available in the library though. Testcases do not seem to fail when a
function is missing for the headers...
The glusterfs-3.7.16 packages have been released with the missing
declaration in the header and symbol-maps. Still, the function is
available for applications:
$ objdump -T usr/lib64/libgfapi.so.0 | grep -w glfs_free
0000000000006aa0 g DF .text 0000000000000035 GFAPI_3.7.16 glfs_free
Cherry picked from commit 5ab0b43ce67d8d131242542a80b6aa6b129ed5ee:
> Change-Id: Ia707ee957f090dbfca028192fcc81a83dfdf4ae0
> BUG: 1344714
> Reported-by: Jiffin Tony Thottan <jthottan@redhat.com>
> Signed-off-by: Niels de Vos <ndevos@redhat.com>
> Reviewed-on: http://review.gluster.org/15653
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
> Smoke: Gluster Build System <jenkins@build.gluster.org>
> Reviewed-by: jiffin tony Thottan <jthottan@redhat.com>
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Change-Id: Ia707ee957f090dbfca028192fcc81a83dfdf4ae0
BUG: 1397506
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/15907
Reviewed-by: soumya k <skoduri@redhat.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'api')
| -rw-r--r-- | api/src/gfapi.aliases | 1 | ||||
| -rw-r--r-- | api/src/gfapi.map | 1 | ||||
| -rw-r--r-- | api/src/glfs.h | 3 |
3 files changed, 5 insertions, 0 deletions
diff --git a/api/src/gfapi.aliases b/api/src/gfapi.aliases index c1331b9d582..6857ab79503 100644 --- a/api/src/gfapi.aliases +++ b/api/src/gfapi.aliases @@ -141,6 +141,7 @@ _priv_glfs_process_upcall_event _glfs_process_upcall_event$GFAPI_PRIVATE_3.7.0 _pub_glfs_h_lookupat _glfs_h_lookupat$GFAPI_3.7.4 +_pub_glfs_free _glfs_free$GFAPI_3.7.16 _pub_glfs_h_poll_upcall _glfs_h_poll_upcall$GFAPI_3.7.16 _pub_glfs_upcall_get_fs _glfs_upcall_get_fs$GFAPI_3.7.16 _pub_glfs_upcall_get_reason _glfs_upcall_get_reason$GFAPI_3.7.16 diff --git a/api/src/gfapi.map b/api/src/gfapi.map index f7c0918fb13..7d2f0227912 100644 --- a/api/src/gfapi.map +++ b/api/src/gfapi.map @@ -175,6 +175,7 @@ GFAPI_3.7.15 { GFAPI_3.7.16 { global: + glfs_free; glfs_upcall_get_fs; glfs_upcall_get_reason; glfs_upcall_inode_get_event; diff --git a/api/src/glfs.h b/api/src/glfs.h index c81b7cc5d0b..903492a9466 100644 --- a/api/src/glfs.h +++ b/api/src/glfs.h @@ -769,6 +769,9 @@ int glfs_posix_lock (glfs_fd_t *fd, int cmd, struct flock *flock) __THROW glfs_fd_t *glfs_dup (glfs_fd_t *fd) __THROW GFAPI_PUBLIC(glfs_dup, 3.4.0); +void glfs_free (void *ptr) __THROW + GFAPI_PUBLIC(glfs_free, 3.7.16); + /* * Nobody needs this call at all yet except for the test script. */ |