diff --git a/expat-alloc-size.patch b/expat-alloc-size.patch
index 0c5fc16..798679a 100644
--- a/expat-alloc-size.patch
+++ b/expat-alloc-size.patch
@@ -2,7 +2,7 @@ Index: expat-2.1.1/lib/expat.h
 ===================================================================
 --- expat-2.1.1.orig/lib/expat.h
 +++ expat-2.1.1/lib/expat.h
-@@ -973,9 +973,13 @@ XML_FreeContentModel(XML_Parser parser,
+@@ -973,9 +973,12 @@ XML_FreeContentModel(XML_Parser parser,
  
  /* Exposing the memory handling functions used in Expat */
  XMLPARSEAPI(void *)
@@ -11,7 +11,6 @@ Index: expat-2.1.1/lib/expat.h
  XML_MemMalloc(XML_Parser parser, size_t size);
  
  XMLPARSEAPI(void *)
-+XML_ATTR_MALLOC
 +XML_ATTR_ALLOC_SIZE(3)
  XML_MemRealloc(XML_Parser parser, void *ptr, size_t size);
  
diff --git a/expat.changes b/expat.changes
index a7d065d..dd94182 100644
--- a/expat.changes
+++ b/expat.changes
@@ -1,3 +1,12 @@
+-------------------------------------------------------------------
+Fri Apr  1 16:32:27 UTC 2016 - crrodriguez@opensuse.org
+
+- After simplification of expat-visibility.patch, it became
+  uneffective as no symbols are getting hidden. add
+  -fvisibility=hidden to CFLAGS again.
+- expat-alloc-size.patch: fix braino, realloc()-like functions
+  should not take __attribute__(malloc)
+
 -------------------------------------------------------------------
 Wed Mar 23 08:31:29 UTC 2016 - idonmez@suse.com
 
diff --git a/expat.spec b/expat.spec
index 2c0ec3f..703406b 100644
--- a/expat.spec
+++ b/expat.spec
@@ -72,6 +72,7 @@ cp %{SOURCE1} .
 rm -f examples/*.dsp
 
 %build
+export CFLAGS="%optflags -fvisibility=hidden"
 %configure --disable-static \
            --with-pic
 make %{?_smp_mflags}