<feed xmlns='http://www.w3.org/2005/Atom'>
<title>samba.git/source3/lib/version.c, branch talloc-2.3.3</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/'/>
<entry>
<title>s3: make better use of ccache by not including version.h in every C-file.</title>
<updated>2009-01-15T21:56:01+00:00</updated>
<author>
<name>Michael Adam</name>
<email>obnox@samba.org</email>
</author>
<published>2009-01-15T21:27:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=39ec8791f874a0a9a853aa1497099a57c1aa1dbc'/>
<id>39ec8791f874a0a9a853aa1497099a57c1aa1dbc</id>
<content type='text'>
version.h changes rather frequently. Since it is included via includes.h,
this means each C file will be a cache miss. This applies to the following
situations:

* When building a new package with a new Samba version

* building in a git branch after calling mkversion.sh
  after a new commit (i.e. virtually always)

This patch improves the situation in the following way:

* remove inlude "version.h" from includes.h

* Use samba_version_string() instead of SAMBA_VERSION_STRING
  in files that use no other macro from version.h instead of
  SAMBA_VERSION_STRING.

* explicitly include "version.h" in those files that use more
  macros from "version.h" than just SAMBA_VERSION_STRING.

Michael
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
version.h changes rather frequently. Since it is included via includes.h,
this means each C file will be a cache miss. This applies to the following
situations:

* When building a new package with a new Samba version

* building in a git branch after calling mkversion.sh
  after a new commit (i.e. virtually always)

This patch improves the situation in the following way:

* remove inlude "version.h" from includes.h

* Use samba_version_string() instead of SAMBA_VERSION_STRING
  in files that use no other macro from version.h instead of
  SAMBA_VERSION_STRING.

* explicitly include "version.h" in those files that use more
  macros from "version.h" than just SAMBA_VERSION_STRING.

Michael
</pre>
</div>
</content>
</entry>
<entry>
<title>Reorder defines for version for consistency with Samba 4.</title>
<updated>2008-09-12T10:38:54+00:00</updated>
<author>
<name>Jelmer Vernooij</name>
<email>jelmer@samba.org</email>
</author>
<published>2008-09-12T10:38:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=41ebc6fe73fc2a2ad845c2ad0109222612edf1a3'/>
<id>41ebc6fe73fc2a2ad845c2ad0109222612edf1a3</id>
<content type='text'>
(This used to be commit fe36d83508a6b1a8de5ad58ec1d0c80cda7b4a21)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
(This used to be commit fe36d83508a6b1a8de5ad58ec1d0c80cda7b4a21)
</pre>
</div>
</content>
</entry>
<entry>
<title>Simplify samba_version_string.</title>
<updated>2008-07-17T12:16:19+00:00</updated>
<author>
<name>Andreas Schneider</name>
<email>anschneider@suse.de</email>
</author>
<published>2008-07-17T10:25:31+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=2a8ebdca535aed431d51270a2bdcafce0b55bb12'/>
<id>2a8ebdca535aed431d51270a2bdcafce0b55bb12</id>
<content type='text'>
This adds a new macro for a vender version function which will be returned
by the samba_version_string() function if defined.

Signed-off-by: Andreas Schneider &lt;anschneider@suse.de&gt;
Signed-off-by: Stefan Metzmacher &lt;metze@samba.org&gt;
(This used to be commit 8b9301fbc73f28c657ec81c605760a0ff7a1973a)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This adds a new macro for a vender version function which will be returned
by the samba_version_string() function if defined.

Signed-off-by: Andreas Schneider &lt;anschneider@suse.de&gt;
Signed-off-by: Stefan Metzmacher &lt;metze@samba.org&gt;
(This used to be commit 8b9301fbc73f28c657ec81c605760a0ff7a1973a)
</pre>
</div>
</content>
</entry>
<entry>
<title>Move samba_extended_info_version to smbd/trans2.c</title>
<updated>2008-01-22T12:00:22+00:00</updated>
<author>
<name>Volker Lendecke</name>
<email>vl@samba.org</email>
</author>
<published>2008-01-22T12:00:22+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=d4d962f691d13f36226157c38f706ee4d10b9150'/>
<id>d4d962f691d13f36226157c38f706ee4d10b9150</id>
<content type='text'>
This is right now only used there, and in version.c it gave linker errors
because some binaries (e.g. smbmnt) don't link in time.o
(This used to be commit 1f0eaaa5911f893c822465a26fe49ab65afb0730)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is right now only used there, and in version.c it gave linker errors
because some binaries (e.g. smbmnt) don't link in time.o
(This used to be commit 1f0eaaa5911f893c822465a26fe49ab65afb0730)
</pre>
</div>
</content>
</entry>
<entry>
<title>Avoid use of uninitialized memory</title>
<updated>2008-01-22T11:46:51+00:00</updated>
<author>
<name>Volker Lendecke</name>
<email>vl@samba.org</email>
</author>
<published>2008-01-22T11:46:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=1d99598ca3f21dd863c8d63698f8af2b3b77759b'/>
<id>1d99598ca3f21dd863c8d63698f8af2b3b77759b</id>
<content type='text'>
(This used to be commit 85123aacdb13e97c3f44aeded1c80e13af53d83d)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
(This used to be commit 85123aacdb13e97c3f44aeded1c80e13af53d83d)
</pre>
</div>
</content>
</entry>
<entry>
<title>Get Samba version or capability information from Windows</title>
<updated>2008-01-22T11:39:47+00:00</updated>
<author>
<name>Corinna Vinschen</name>
<email>corinna@vinschen.de</email>
</author>
<published>2008-01-22T10:33:17+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=559603ad693ede382c566cb8bbd8e6bd8d5750a6'/>
<id>559603ad693ede382c566cb8bbd8e6bd8d5750a6</id>
<content type='text'>
On Jan 21 16:18, Danilo Almeida wrote:
&gt; Corina wrote:
&gt;
&gt; &gt; +	time_t samba_gitcommitdate;
&gt;
&gt; And:
&gt;
&gt; &gt; +   SIVAL(pdata,28,extended_info.samba_gitcommitdate);
&gt; &gt; +   memcpy(pdata+32,extended_info.samba_version_string,32);
&gt;
&gt; Note that you are dropping bits on a system w/64-bit time_t, and that this has the 2038 problem.

Right.  I changed samba_gitcommitdate from time_t to NTTIME and shortened
samba_version_string to 28 bytes.  New patch below.

Thanks,
Corinna
(This used to be commit 28aa1c199d3a22cda34afcaab49c0561eeb0abcb)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
On Jan 21 16:18, Danilo Almeida wrote:
&gt; Corina wrote:
&gt;
&gt; &gt; +	time_t samba_gitcommitdate;
&gt;
&gt; And:
&gt;
&gt; &gt; +   SIVAL(pdata,28,extended_info.samba_gitcommitdate);
&gt; &gt; +   memcpy(pdata+32,extended_info.samba_version_string,32);
&gt;
&gt; Note that you are dropping bits on a system w/64-bit time_t, and that this has the 2038 problem.

Right.  I changed samba_gitcommitdate from time_t to NTTIME and shortened
samba_version_string to 28 bytes.  New patch below.

Thanks,
Corinna
(This used to be commit 28aa1c199d3a22cda34afcaab49c0561eeb0abcb)
</pre>
</div>
</content>
</entry>
<entry>
<title>Use SAFE_FREE instead of free</title>
<updated>2008-01-19T18:31:02+00:00</updated>
<author>
<name>Volker Lendecke</name>
<email>vl@samba.org</email>
</author>
<published>2008-01-19T18:31:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=07e07f696ad10f3a5d7a0d9de656ff13600ac97d'/>
<id>07e07f696ad10f3a5d7a0d9de656ff13600ac97d</id>
<content type='text'>
(This used to be commit 999647329028147d7c29a3348202641b3e03430e)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
(This used to be commit 999647329028147d7c29a3348202641b3e03430e)
</pre>
</div>
</content>
</entry>
<entry>
<title>Tiny memory leak in lib/version.c</title>
<updated>2008-01-19T18:30:16+00:00</updated>
<author>
<name>Corinna Vinschen</name>
<email>corinna@vinschen.de</email>
</author>
<published>2008-01-19T14:14:45+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=a0bd9d97a3a701fdb1f9a48ce925f63b786a8070'/>
<id>a0bd9d97a3a701fdb1f9a48ce925f63b786a8070</id>
<content type='text'>
Hi,

while implementing the extra_info version stuff, it occured to me that
samba_version_string() potentially allocates memory which is unused but
never free'd.

If SAMBA_VERSION_VENDOR_PATCH is defined, a second call to asprintf
takes place.  The result is stored in tmp_version.  Afterwards,
samba_version is set to tmp_version without free'ing samba_version
first.  Looks like a simple free(samba_version) is missing.  Patch
against 3.2-test below.

Ok, this only happens once over the lifetime of the application, so it's
no big deal, but I though it doesn't hurt to mention it.

Corinna

	* lib/version.c (samba_version_string): Free samba_version
	before setting to tmp_version.
(This used to be commit 373a23d48f2dd24e65dbf814ea58b4add2322128)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Hi,

while implementing the extra_info version stuff, it occured to me that
samba_version_string() potentially allocates memory which is unused but
never free'd.

If SAMBA_VERSION_VENDOR_PATCH is defined, a second call to asprintf
takes place.  The result is stored in tmp_version.  Afterwards,
samba_version is set to tmp_version without free'ing samba_version
first.  Looks like a simple free(samba_version) is missing.  Patch
against 3.2-test below.

Ok, this only happens once over the lifetime of the application, so it's
no big deal, but I though it doesn't hurt to mention it.

Corinna

	* lib/version.c (samba_version_string): Free samba_version
	before setting to tmp_version.
(This used to be commit 373a23d48f2dd24e65dbf814ea58b4add2322128)
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove a static fstring</title>
<updated>2007-12-16T13:02:26+00:00</updated>
<author>
<name>Volker Lendecke</name>
<email>vl@samba.org</email>
</author>
<published>2007-12-15T23:03:56+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=dd4e99c9d7111fe1fab9b0a968c1321425f83429'/>
<id>dd4e99c9d7111fe1fab9b0a968c1321425f83429</id>
<content type='text'>
(This used to be commit a9c62c57db9e580640d0265b08b3178496de76a8)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
(This used to be commit a9c62c57db9e580640d0265b08b3178496de76a8)
</pre>
</div>
</content>
</entry>
<entry>
<title>Correct VERSION and version.c to use SAMBA_VERSION_VENDOR_PATCH</title>
<updated>2007-10-21T17:32:06+00:00</updated>
<author>
<name>Gerald (Jerry) Carter</name>
<email>jerry@samba.org</email>
</author>
<published>2007-10-21T17:32:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.exis.tech/samba.git/commit/?id=69c6f16e31d0adfc7efb077e09afbb8ff5fb2b0f'/>
<id>69c6f16e31d0adfc7efb077e09afbb8ff5fb2b0f</id>
<content type='text'>
which is used by mkversion.sh.
(This used to be commit a62c7f9350322ab435244eafe6053fd5ee618973)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
which is used by mkversion.sh.
(This used to be commit a62c7f9350322ab435244eafe6053fd5ee618973)
</pre>
</div>
</content>
</entry>
</feed>
