Author: gahr (ports committer)
Date: Mon Jul 28 07:20:22 2014
New Revision: 269179
URL: http://svnweb.freebsd.org/changeset/base/269179

Log:
  Unbreak the ABI by reverting r268494 until the compat shims are provided

Modified:
  head/lib/libsbuf/Makefile
  head/sys/kern/subr_sbuf.c
  head/sys/sys/sbuf.h

Modified: head/lib/libsbuf/Makefile
==============================================================================
--- head/lib/libsbuf/Makefile   Mon Jul 28 05:37:10 2014        (r269178)
+++ head/lib/libsbuf/Makefile   Mon Jul 28 07:20:22 2014        (r269179)
@@ -4,7 +4,7 @@ LIB=            sbuf
 SHLIBDIR?=     /lib
 SRCS=          subr_sbuf.c 
 
-SHLIB_MAJOR    = 7
+SHLIB_MAJOR    = 6
 
 SYMBOL_MAPS=    ${.CURDIR}/Symbol.map
 VERSION_DEF=   ${.CURDIR}/Version.def

Modified: head/sys/kern/subr_sbuf.c
==============================================================================
--- head/sys/kern/subr_sbuf.c   Mon Jul 28 05:37:10 2014        (r269178)
+++ head/sys/kern/subr_sbuf.c   Mon Jul 28 07:20:22 2014        (r269179)
@@ -152,16 +152,11 @@ static int
 sbuf_extend(struct sbuf *s, int addlen)
 {
        char *newbuf;
-       size_t newsize;
+       int newsize;
 
        if (!SBUF_CANEXTEND(s))
                return (-1);
        newsize = sbuf_extendsize(s->s_size + addlen);
-       if (s->s_buf == s->s_static_buf && newsize <= sizeof(s->s_static_buf)) {
-               s->s_size = sizeof(s->s_static_buf);
-               return (0);
-       }
-
        newbuf = SBMALLOC(newsize);
        if (newbuf == NULL)
                return (-1);
@@ -181,7 +176,7 @@ sbuf_extend(struct sbuf *s, int addlen)
  * big enough to hold at least length characters.
  */
 static struct sbuf *
-sbuf_newbuf(struct sbuf *s, char *buf, size_t length, int flags)
+sbuf_newbuf(struct sbuf *s, char *buf, int length, int flags)
 {
 
        memset(s, 0, sizeof(*s));
@@ -200,11 +195,6 @@ sbuf_newbuf(struct sbuf *s, char *buf, s
        if ((flags & SBUF_AUTOEXTEND) != 0)
                s->s_size = sbuf_extendsize(s->s_size);
 
-       if (s->s_size <= sizeof(s->s_static_buf)) {
-               s->s_buf = s->s_static_buf;
-               return (s);
-       }
-
        s->s_buf = SBMALLOC(s->s_size);
        if (s->s_buf == NULL)
                return (NULL);
@@ -293,19 +283,21 @@ sbuf_clear(struct sbuf *s)
  * Effectively truncates the sbuf at the new position.
  */
 int
-sbuf_setpos(struct sbuf *s, size_t pos)
+sbuf_setpos(struct sbuf *s, ssize_t pos)
 {
 
        assert_sbuf_integrity(s);
        assert_sbuf_state(s, 0);
 
+       KASSERT(pos >= 0,
+           ("attempt to seek to a negative position (%jd)", (intmax_t)pos));
        KASSERT(pos < s->s_size,
            ("attempt to seek past end of sbuf (%jd >= %jd)",
            (intmax_t)pos, (intmax_t)s->s_size));
        KASSERT(!SBUF_ISSECTION(s),
            ("attempt to seek when in a section"));
 
-       if (pos > s->s_len)
+       if (pos < 0 || pos > s->s_len)
                return (-1);
        s->s_len = pos;
        return (0);
@@ -569,8 +561,7 @@ int
 sbuf_vprintf(struct sbuf *s, const char *fmt, va_list ap)
 {
        va_list ap_copy;
-       size_t len;
-       int error;
+       int error, len;
 
        assert_sbuf_integrity(s);
        assert_sbuf_state(s, 0);

Modified: head/sys/sys/sbuf.h
==============================================================================
--- head/sys/sys/sbuf.h Mon Jul 28 05:37:10 2014        (r269178)
+++ head/sys/sys/sbuf.h Mon Jul 28 07:20:22 2014        (r269179)
@@ -44,8 +44,8 @@ struct sbuf {
        sbuf_drain_func *s_drain_func;  /* drain function */
        void            *s_drain_arg;   /* user-supplied drain argument */
        int              s_error;       /* current error code */
-       size_t           s_size;        /* size of storage buffer */
-       size_t           s_len;         /* current length of string */
+       ssize_t          s_size;        /* size of storage buffer */
+       ssize_t          s_len;         /* current length of string */
 #define        SBUF_FIXEDLEN   0x00000000      /* fixed length buffer 
(default) */
 #define        SBUF_AUTOEXTEND 0x00000001      /* automatically extend buffer 
*/
 #define        SBUF_USRFLAGMSK 0x0000ffff      /* mask of flags the user may 
specify */
@@ -55,9 +55,6 @@ struct sbuf {
 #define        SBUF_INSECTION  0x00100000      /* set by sbuf_start_section() 
*/
        int              s_flags;       /* flags */
        ssize_t          s_sect_len;    /* current length of section */
-#define SBUF_STATIC_LEN        64              /* static storage buffer length 
*/
-       char             s_static_buf[SBUF_STATIC_LEN];
-                                       /* static storage buffer */
 };
 
 __BEGIN_DECLS
@@ -68,7 +65,7 @@ struct sbuf   *sbuf_new(struct sbuf *, cha
 #define                 sbuf_new_auto()                                \
        sbuf_new(NULL, NULL, 0, SBUF_AUTOEXTEND)
 void            sbuf_clear(struct sbuf *);
-int             sbuf_setpos(struct sbuf *, size_t);
+int             sbuf_setpos(struct sbuf *, ssize_t);
 int             sbuf_bcat(struct sbuf *, const void *, size_t);
 int             sbuf_bcpy(struct sbuf *, const void *, size_t);
 int             sbuf_cat(struct sbuf *, const char *);
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to