AtlServer Work Item Rss Feed

Commented Issue: SPROXY ignores minOccurs="0" [4118]

icnocop — Thu, 11 Apr 2013 01:18:10 GMT

Currently the stub generation does not take the special case of minOccurs="0" into consideration. It should treat these items the same as if the wsdl has specified nillable="true" as per Trevors previous fix.

Gordon
Comments: ** Comment from web user: icnocop **

Schmoo has uploaded [patch #456](https://atlserver.codeplex.com/SourceControl/list/patches) for this issue.

Commented Issue: Make ATLPref work under Vista [6815]

icnocop — Thu, 11 Apr 2013 01:09:14 GMT

The pref objects don't work under Vista, there seem to be two problems with it:
Comments: ** Comment from web user: icnocop **

There is a duplicate issue here: [Perfomance Counter Support in Vista](https://atlserver.codeplex.com/workitem/9279)

Commented Issue: Perfomance Counter Support in Vista [9279]

icnocop — Thu, 11 Apr 2013 01:08:54 GMT

The performance counter support in Vista is broken.
Comments: ** Comment from web user: icnocop **

See this related issue: [Make ATLPref work under Vista](https://atlserver.codeplex.com/workitem/6815)

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

AlanChen — Thu, 24 May 2012 10:02:52 GMT

I see atlperf.h creates a section as readonly-shared. However I don't understand why is it needed. I've dumpbin'ed my dll compiled with vs2005 and it looks like it was always 40000040, just with no warning.
If I specify /section:.rdata,rs for linker it give another warning: warning LNK4223: the SHARED (S) attribute specified for section '.rdata' is ineffective without the WRITE (W) attribute

Is the "shared" really needed for ATLP section?
Comments: ** Comment from web user: AlanChen **

Just Ignore it , nobody fix this!

property: linker: commandline--> Ignore:4254

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

research_opencs — Mon, 07 Nov 2011 16:31:01 GMT

I have a similar problem with "ATLS" section and VC2010.

Based on the comments posted by vvdb I discovered that this error is caused because the .rdata in VC2010 does not accept the attribute shared required by ATLS (dumpbin):

SECTION HEADER #3
.rdata name
2651D virtual size
FC000 virtual address (100FC000 to 1012251C)
26600 size of raw data
AA000 file pointer to raw data (000AA000 to 000D05FF)
0 file pointer to relocation table
0 file pointer to line numbers
0 number of relocations
0 number of line numbers
40000040 flags
Initialized Data
Read Only

While the ATLS defines the attribute shared:

SECTION HEADER #6
ATLS name
208 virtual size
12B000 virtual address (1012B000 to 1012B207)
400 size of raw data
D3E00 file pointer to raw data (000D3E00 to 000D41FF)
0 file pointer to relocation table
0 file pointer to line numbers
0 number of relocations
0 number of line numbers
50000040 flags
Initialized Data
Shared
Read Only

In order to solve this, I modified the header "atlisapi.h" to remove the merge if _MSC_VER is greater or equal to 1600 (VC2010). This fixed the problem and appears to be working with no major problems except for the existence of an extra segment.

The modification is as follows:

#if !defined(_M_IA64)
#if _MSC_VER < 1600
#pragma comment(linker, "/merge:ATLS=.rdata")
#endif
#endif

It is possible that the same approach will solve the error detected with the ATL section as well because the problem appears to be the same.

More details about the sections can be found in http://msdn.microsoft.com/en-us/library/sf9b18xk.aspx

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

pally_sandher — Fri, 19 Aug 2011 16:25:59 GMT

Still seeing this with SP1 for VS2010 applied.

Commented Issue: Possible crash in CAtlHttpClientT [15897]

asalamon — Fri, 20 May 2011 18:57:57 GMT

If a binary file is being downloaded using chunked Transfer-Encoding, and the binary file contains a null character, this line in ReadChunkedBody will only append some of the result_buffer to m_current, because Append with no length uses strlen, which will stop at the null. Later attempts to use m_current can cause access violations because GetBodyLength() still returns the actual download size.
if (!m_current.Append((LPCSTR)result_buffer))
return false;

Changing it to this, fixes the problem:
if ( !m_current.Append((LPCSTR)result_buffer, result_buffer.GetLength()) )
return false;

Not a bug, but I also added this code in ReadChunkedBody just after a successful Read so that callback clients will get feedback during a chunked download.
if (m_pNavData->pfnReadStatusCallback)
{
bool bRet = m_pNavData->pfnReadStatusCallback(dwReadBuffSize, m_pNavData->m_lParamRead);
if (!bRet)
return bRet;
}
Comments: ** Comment from web user: asalamon **

Apparently this was reported as far back as 2002 and still hasn't been fixed:
http://groups.google.com/group/microsoft.public.vc.atl/msg/4e2e29008d33ca17

Created Issue: Possible crash in CAtlHttpClientT [15897]

asalamon — Fri, 20 May 2011 18:34:09 GMT

Commented Issue: Make ATLPref work under Vista [6815]

audriusz — Tue, 28 Dec 2010 15:14:04 GMT

The pref objects don't work under Vista, there seem to be two problems with it:
Comments: ** Comment from web user: audriusz **

I had encountered supposedly the same issue on Windows Server 2008 R2. I believe that this is the result of a problem inside RegQueryValueEx, by including uninitialized data into the reported byte array. As with other uninitialized data, reproducing the problem is quite random.
I have submitted the issue via Microsoft Connect (https://connect.microsoft.com/WindowsServerFeedback/feedback/details/633595/a-possible-problem-in-winapi-function-regqueryvalueex-prevents-perf-counter-registration#)

A simple workaround may be coded into ATL Server, <atlperf.inl>:
Metods CPerfMon::_AppendRegStrings and CPerfMon::_RemoveRegStrings should zero-fill the buffer inside CStrBuf passed as a parameter to the second calls to rkLang.QueryMultiStringValue.

Commented Issue: CSoapHandler not wired under IIS7, Win Server 2008 [14847]

boolka — Wed, 08 Dec 2010 21:16:47 GMT

Running against IIS7 / Windows Server 2008, CSoapHandler Isapi extension would not route calls into the instance of the soap handler due to a bug in atlisapi.h

The fix -- inside class CServerContext had to change implementation of the Initialize method to obtain the script path explicitly

void Initialize(__in EXTENSION_CONTROL_BLOCK *pECB)
{
ATLENSURE(pECB);
m_pECB = pECB;

// Initialize the translated script path
DWORD dwSize=0;
BOOL bRes = GetServerVariable("SCRIPT_TRANSLATED", NULL, &dwSize);
char *szScriptPathTranslated = NULL;
ATLTRY(szScriptPathTranslated = m_strScriptPathTranslated.GetBuffer(dwSize));
if (szScriptPathTranslated)
{
GetServerVariable("SCRIPT_TRANSLATED", szScriptPathTranslated, &dwSize);
szScriptPathTranslated[dwSize] = '\0';
m_strScriptPathTranslated.ReleaseBuffer(dwSize);
}

// Old implementation:
//_AtlGetScriptPathTranslated(GetPathTranslated(), m_strScriptPathTranslated);
}
Comments: ** Comment from web user: boolka **

The above is inside atlisapi.h.

Created Issue: CSoapHandler not wired under IIS7, Win Server 2008 [14847]

boolka — Wed, 08 Dec 2010 21:15:10 GMT

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

mloskot — Wed, 01 Dec 2010 15:54:28 GMT

It looks to me there is conflict between read-only and shared attributes of the declared sections and merging them with .rdata.

I have asked related question here http://social.msdn.microsoft.com/Forums/en-US/vcmfcatl/thread/4763bc2d-0e86-45e9-bfe4-ddaa735e5ad5

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

Paul7 — Mon, 18 Oct 2010 23:24:39 GMT

The same issue here. Lots of warnings when linking. This could be a problem with VC ATL includes. In VS 2008 atlbase.h the part of the file with pragmas is compiled conditionally.
$if defined(_M_IA64) || defined(_M_IX86) ...
In VS 2010 the same declaration in atlbase.h is not conditional so it is included in 64 bit compilation.
For now I prefer to ignore these warnings. Maybe in the next VS 2010 service pack it will be fixed.

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

ReenaG — Tue, 15 Jun 2010 00:13:10 GMT

I am having the same problem. Since this is a warning and not an error is it Ok to ignore it?

Commented Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

vvdb — Tue, 04 May 2010 11:25:15 GMT

I get the same linker warning here.
It's not clear how to solve this problem.
Probably due to the following:

#if !defined(_M_IA64)
#pragma comment(linker, "/merge:ATL=.rdata")
#endif

being defined in the standard ATL files, but NOT in the ATLServer files.

I tried adding it around i.e. #include <atlhttp.h>, but it doesn't help.

Any help in solving this problem is greatly appreciated.

Created Issue: DllMain rule violation in _g_smtp_init's constructor [12923]

denissie — Mon, 03 May 2010 11:59:23 GMT

In the atlsmtpconnection.h file, the global variable declaration "__declspec(selectany) CSMTPWSAStartup _g_smtp_init;" calls "WSAStartup(ATLSMTP_WSA_VERSION, &wsadata)" in its constructor (see init() method).
This violates the rule that we may not call imported functions other than those located in Kernel32.dll (see "DllMain Restrictions" section at http://msdn.microsoft.com/en-us/library/aa290048%28VS.71%29.aspx).

Possible solutions (IMHO):
1) remove the init() call from the constructor of CSMTPWSAStartup
2) wrap the _g_smtp_init object in a songleton

Note:
Because of this issue, we had a crash when the DLLS were loaded (DllMain) -> hard to diagnose & debug. DLL Delay Load (in Visual Studio) helped, but it isn't a "real" solution...

Created Issue: vs2010: warning LNK4254: section 'ATLP' (50000040) merged into '.rdata' (40000040) with different attributes [12158]

JarLob — Sat, 20 Mar 2010 17:14:36 GMT

Created Issue: atlperf.inl generates PreFast static analyzer warnings: C6284 objects passed, when string is required. [10641]

vbaderks — Thu, 17 Sep 2009 11:29:28 GMT

At several locations in atlper.inl an object is passed while a string is required.

For example:

strMutexName.Format(_T("Global\\ATLPERF_%s_LOCK"), strAppName); (line 1873)

This should be:

strMutexName.Format(_T("Global\\ATLPERF_%s_LOCK"), strAppName.GetString());

Commented Issue: ATLMIME.H - GetRecipientsString()

pawelek — Fri, 20 Feb 2009 12:28:28 GMT

Version: ATL Server Source and Headers 9.0.70425 Alpha

Problem:
New code added to used function AtlMimeMakeRecipientsString() means that GetRecipientsString() will always return FALSE.
The problem is that the 'dwLen' parameter will always be zero, and buffer length validation in AtlMimeMakeRecipientsString() will fail the call.

Fix:
In GetRecipientsString() the three instances of this line:
dwLen = *pdwLen - dwTotalLen;
should be changed to this:
dwLen = dwMaxLen - dwTotalLen;
This will correctly maintain the remaining buffer length in dwLen.
Comments: ** Comment from web user: pawelek **

You can simply comment line 531 of the ATLMIME.H like this:

530: DWORD dwMaxLen = *pdwLen;
531: //*pdwLen = 0;

This is a serious error and it should be corrected in the next release.

Created Issue: Perfomance Counter Support in Vista

mireku — Thu, 19 Feb 2009 05:01:08 GMT

The performance counter support in Vista is broken.