pdump_file_get (src/dumper.c)

older

xemacs-21.2.38

November sumo release postponed.

Hiroaki Abe

Sunday, 3 December 2000 Sun, 3 Dec '00

3:18 p.m.

(permalink)

阿部と申します。始めまして。 21.2.37のsrc/dumper.cをみていて気が付いたのですが、pdump_file_get()中に出てくる pdump_fd は fd だと思うのですが？ diff -ur xemacs-21.2.37.org/src/dumper.c xemacs-21.2.37/src/dumper.c --- xemacs-21.2.37.org/src/dumper.c Tue Oct 3 08:27:01 2000 +++ xemacs-21.2.37/src/dumper.c Sun Dec 3 16:52:56 2000 ＠＠ -1238,9 +1238,9 ＠＠ pdump_mallocadr = xmalloc(pdump_length+255); pdump_free = pdump_file_free; pdump_start = (char *)((255 + (unsigned long)pdump_mallocadr) & ~255); - read (pdump_fd, pdump_start, pdump_length); + read (fd, pdump_start, pdump_length); - close (pdump_fd); + close (fd); return 1; } #endif /* !WIN32_NATIVE */ -- Hiroaki Abe

Show replies by date

Yoshiki Hayashi

Monday, 4 December Mon, 4 Dec

4:01 a.m.

(permalink)

Hiroaki Abe <h-abe(a)pc.highway.ne.jp> writes:

...

21.2.37のsrc/dumper.cをみていて気が付いたのですが、pdump_file_get()中に出てくる pdump_fd は fd だと思うのですが？

どうもそのようですね。xemacs-patches に出して頂けないでしょうか。dumper だと Olivier さんが反応してくれると思います。 -- Yoshiki Hayashi

Stephen J. Turnbull

Monday, 4 December Mon, 4 Dec

4:14 a.m.

(permalink)

...

>>>> "Yoshiki" == Yoshiki Hayashi <yoshiki(a)xemacs.org> writes:

Yoshiki> Hiroaki Abe <h-abe(a)pc.highway.ne.jp> writes:

...

> 21.2.37のsrc/dumper.cをみていて気が付いたのですが、 > pdump_file_get()中に出てくる pdump_fd は fd だと思うのですが？

Yoshiki> どうもそのようですね。xemacs-patches に出して頂けないでしょ Yoshiki> うか。dumper だと Olivier さんが反応してくれると思います。 l.176 を参照してください。 static int pdump_fd; -- University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN Institute of Policy and Planning Sciences Tel/fax: +81 (298) 53-5091 _________________ _________________ _________________ _________________ What are those straight lines for? "XEmacs rules."

Stephen J. Turnbull

Monday, 4 December Mon, 4 Dec

5:28 a.m.

(permalink)

...

>>>> "Yoshiki" == Yoshiki Hayashi <yoshiki(a)xemacs.org> writes:

Yoshiki> Hiroaki Abe <h-abe(a)pc.highway.ne.jp> writes:

...

>> 21.2.37のsrc/dumper.cをみていて気が付いたのですが、 >> pdump_file_get()中に出てくる pdump_fd は fd だと思うのですが？

Yoshiki> どうもそのようですね。xemacs-patches に出して頂けないでしょ Yoshiki> うか。dumper だと Olivier さんが反応してくれると思います。

...

>>>> "Stephen" == Stephen J Turnbull <turnbull(a)sk.tsukuba.ac.jp> writes:

Stephen> l.176 を参照してください。 Stephen> static int pdump_fd; Oops. 以下の部分は省略してしまったのです。 dumper.c のなかでは pdump_file_get は load time しか使わないのですが load time には pdump_fd は open されていません。逆に fd は close されないので pdump_fd -> fd のパッチは正しいと思います。提出するときにそういう説明を加えるといいでしょう。 -- University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN Institute of Policy and Planning Sciences Tel/fax: +81 (298) 53-5091 _________________ _________________ _________________ _________________ What are those straight lines for? "XEmacs rules."

Hiroaki Abe

Monday, 4 December Mon, 4 Dec

12:24 p.m.

(permalink)

The following patch is for src/dumper.c. It seems that pdump_fd isn't used when pdump_file_get() is called, so read() fails and fd isn't closed. diff -ur xemacs-21.2.37.org/src/dumper.c xemacs-21.2.37/src/dumper.c --- xemacs-21.2.37.org/src/dumper.c Tue Oct 3 08:27:01 2000 +++ xemacs-21.2.37/src/dumper.c Sun Dec 3 16:52:56 2000 ＠＠ -1238,9 +1238,9 ＠＠ pdump_mallocadr = xmalloc(pdump_length+255); pdump_free = pdump_file_free; pdump_start = (char *)((255 + (unsigned long)pdump_mallocadr) & ~255); - read (pdump_fd, pdump_start, pdump_length); + read (fd, pdump_start, pdump_length); - close (pdump_fd); + close (fd); return 1; } #endif /* !WIN32_NATIVE */ -- Hiroaki Abe

Olivier Galibert

Monday, 4 December Mon, 4 Dec

6:46 p.m.

(permalink)

On Mon, Dec 04, 2000 at 09:24:05PM +0900, Hiroaki Abe wrote:

...

The following patch is for src/dumper.c. It seems that pdump_fd isn't used when pdump_file_get() is called, so read() fails and fd isn't closed.

Reviewed, approved and committed. Thanks. OG. PS: It would be nice if you could add a Changelog next time :-)

8823

days inactive

8824

days old

xemacs-beta-ja@xemacs.org

Manage subscription

5 comments

4 participants

tags (0)

participants (4)

Hiroaki Abe
Olivier Galibert
Stephen J. Turnbull
Yoshiki Hayashi

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

2000

1999

pdump_file_get (src/dumper.c)