]> git.ipfire.org Git - thirdparty/glibc.git/blobdiff - nis/nis_file.c
Adjust wording of two fall-through comments.
[thirdparty/glibc.git] / nis / nis_file.c
index 002e72ed201720b9e4646c5ece929dabed27c11f..9f9e33eb663a161172412adb0c3b0e478be7b846 100644 (file)
-/* Copyright (c) 1997 Free Software Foundation, Inc.
+/* Copyright (c) 1997-2019 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
-   Contributed by Thorsten Kukuk <kukuk@vt.uni-paderborn.de>, 1997.
+   Contributed by Thorsten Kukuk <kukuk@suse.de>, 1997.
 
    The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Library General Public License as
-   published by the Free Software Foundation; either version 2 of the
-   License, or (at your option) any later version.
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
 
    The GNU C Library is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Library General Public License for more details.
+   Lesser General Public License for more details.
 
-   You should have received a copy of the GNU Library General Public
-   License along with the GNU C Library; see the file COPYING.LIB.  If not,
-   write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
 
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <rpcsvc/nis.h>
-#include <rpcsvc/nislib.h>
+#include <shlib-compat.h>
+#include "nis_xdr.h"
+
+typedef bool_t (*iofct_t) (XDR *, void *);
+typedef void (*freefct_t) (void *);
 
-directory_obj *
-readColdStartFile (void)
-{
-  XDR xdrs;
-  FILE *in;
-  directory_obj obj;
 
-  in = fopen ("/var/nis/NIS_COLD_START", "rb");
+static void *
+read_nis_obj (const char *name, iofct_t readfct, freefct_t freefct,
+             size_t objsize)
+{
+  FILE *in = fopen (name, "rce");
   if (in == NULL)
+    return NULL;
+
+  void *obj = calloc (1, objsize);
+
+  if (obj != NULL)
     {
-      fputs (_("Error: Could not open /var/nis/NIS_COLD_START!\n"), stdout);
-      return NULL;
-    }
-  memset (&obj, '\0', sizeof (obj));
-  xdrstdio_create (&xdrs, in, XDR_DECODE);
-  if (!xdr_directory_obj (&xdrs, &obj))
-    {
-      fputs (("Error while reading /var/nis/NIS_COLD_START!\n"), stdout);
-      return NULL;
+      XDR xdrs;
+      xdrstdio_create (&xdrs, in, XDR_DECODE);
+      bool_t status = readfct (&xdrs, obj);
+      xdr_destroy (&xdrs);
+
+      if (!status)
+       {
+         freefct (obj);
+         obj = NULL;
+       }
     }
 
-  return nis_clone_directory (&obj, NULL);
+  fclose (in);
+
+  return obj;
 }
 
-bool_t
-writeColdStartFile (const directory_obj *obj)
+static bool_t
+write_nis_obj (const char *name, const void *obj, iofct_t writefct)
 {
-  XDR xdrs;
-  FILE *out;
-
-  out = fopen ("/var/nis/NIS_COLD_START", "wb");
+  FILE *out = fopen (name, "wce");
   if (out == NULL)
     return FALSE;
 
+  XDR xdrs;
   xdrstdio_create (&xdrs, out, XDR_ENCODE);
-  /* XXX The following cast is bad!  Shouldn't the XDR functions take
-     pointers to const objects?  */
-  if (!xdr_directory_obj (&xdrs, (directory_obj *) obj))
-    {
-      fputs (_("Error while reading /var/nis/NIS_COLD_START!\n"), stdout);
-      return FALSE;
-    }
+  bool_t status = writefct (&xdrs, (void *) obj);
+  xdr_destroy (&xdrs);
+  fclose (out);
 
-  return TRUE;
+  return status;
 }
 
-nis_object *
-nis_read_obj (const char *name)
-{
-  XDR xdrs;
-  FILE *in;
-  nis_object obj;
-
-  in = fopen (name, "rb");
-  if (in == NULL)
-    return NULL;
 
-  memset (&obj, '\0', sizeof (obj));
-  xdrstdio_create (&xdrs, in, XDR_DECODE);
-  if (!xdr_nis_object (&xdrs, &obj))
-    return NULL;
+static const char cold_start_file[] = "/var/nis/NIS_COLD_START";
 
-  return nis_clone_object (&obj, NULL);
+directory_obj *
+readColdStartFile (void)
+{
+  return read_nis_obj (cold_start_file, (iofct_t) _xdr_directory_obj,
+                      (freefct_t) nis_free_directory, sizeof (directory_obj));
 }
+libnsl_hidden_nolink_def (readColdStartFile, GLIBC_2_1)
 
 bool_t
-nis_write_obj (const char *name, const nis_object *obj)
+writeColdStartFile (const directory_obj *obj)
 {
-  XDR xdrs;
-  FILE *out;
-
-  out = fopen (name, "wb");
-  if (out == NULL)
-    return FALSE;
+  return write_nis_obj (cold_start_file, obj, (iofct_t) _xdr_directory_obj);
+}
+libnsl_hidden_nolink_def (writeColdStartFile, GLIBC_2_1)
 
-  xdrstdio_create (&xdrs, out, XDR_ENCODE);
-  /* XXX The following cast is bad!  Shouldn't the XDR functions take
-     pointers to const objects?  */
-  if (!xdr_nis_object (&xdrs, (nis_object *) obj))
-    return FALSE;
+nis_object *
+nis_read_obj (const char *name)
+{
+  return read_nis_obj (name, (iofct_t) _xdr_nis_object,
+                      (freefct_t) nis_free_object, sizeof (nis_object));
+}
+libnsl_hidden_nolink_def (nis_read_obj, GLIBC_2_1)
 
-  return TRUE;
+bool_t
+nis_write_obj (const char *name, const nis_object *obj)
+{
+  return write_nis_obj (name, obj, (iofct_t) _xdr_nis_object);
 }
+libnsl_hidden_nolink_def (nis_write_obj, GLIBC_2_1)