diff options
| author | Niels de Vos <ndevos@redhat.com> | 2016-10-19 12:33:35 +0200 | 
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2016-10-21 05:32:14 -0700 | 
| commit | a5bf3c3a586ccf20f5f735968ec9a56d33f62326 (patch) | |
| tree | 8ba925d5aaf3deb345c7385f240aed99b3772bd4 /api/src/glfs.h | |
| parent | 5bc41576647dd7f0e201e7e2951cf75702d8b623 (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: 1347717
Reported-by: Jiffin Tony Thottan <jthottan@redhat.com>
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/15680
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: jiffin tony Thottan <jthottan@redhat.com>
Reviewed-by: Oleksandr Natalenko <oleksandr@natalenko.name>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Diffstat (limited to 'api/src/glfs.h')
| -rw-r--r-- | api/src/glfs.h | 3 | 
1 files changed, 3 insertions, 0 deletions
diff --git a/api/src/glfs.h b/api/src/glfs.h index b53af35b158..8b5782ef084 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); +  /*   * No xdata support for now.  Nobody needs this call at all yet except for the   * test script, and that doesn't need xdata.  Adding dict_t support and a new  | 
