diff mbox

tools/xenstore: define off_t

Message ID 20170303085210.11258-1-olaf@aepfle.de (mailing list archive)
State New, archived
Headers show

Commit Message

Olaf Hering March 3, 2017, 8:52 a.m. UTC 
talloc.h uses off_t, but did not include <sys/types.h>
Fixes 7012548e0d ("xenstore: enhance control command support")

Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
 tools/xenstore/talloc.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Wei Liu March 3, 2017, 9:07 a.m. UTC | #1 
On Fri, Mar 03, 2017 at 08:52:09AM +0000, Olaf Hering wrote:
> talloc.h uses off_t, but did not include <sys/types.h>
> Fixes 7012548e0d ("xenstore: enhance control command support")

I think this issue is exposed by that particular commit. That commit is
fine in itself. So I would simply delete the "Fixes ..." part.


> 
> Signed-off-by: Olaf Hering <olaf@aepfle.de>

Acked-by: Wei Liu <wei.liu2@citrix.com>
diff mbox

Patch

diff --git a/tools/xenstore/talloc.h b/tools/xenstore/talloc.h
index c849bf61af..71a36e7be0 100644
--- a/tools/xenstore/talloc.h
+++ b/tools/xenstore/talloc.h
@@ -24,6 +24,8 @@ 
    License along with this library; If not, see <http://www.gnu.org/licenses/>.
 */
 
+#include <sys/types.h>
+
 /* this is only needed for compatibility with the old talloc */
 typedef void TALLOC_CTX;