Skip to content

Commit

Permalink
gfapi: add glfs_free() to glfs.h
Browse files Browse the repository at this point in the history
Commit 4721188 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

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>
  • Loading branch information
nixpanic authored and Jeff Darcy committed Oct 18, 2016
1 parent bac4247 commit 5ab0b43
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions api/src/gfapi.aliases
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
1 change: 1 addition & 0 deletions api/src/gfapi.map
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
3 changes: 3 additions & 0 deletions api/src/glfs.h
Original file line number Diff line number Diff line change
Expand Up @@ -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.
*/
Expand Down

0 comments on commit 5ab0b43

Please sign in to comment.